Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

✨Needed feature in marsha #155

Open
7 of 14 tasks
AntoLC opened this issue Sep 4, 2023 · 9 comments
Open
7 of 14 tasks

✨Needed feature in marsha #155

AntoLC opened this issue Sep 4, 2023 · 9 comments
Labels
feature request New feature or request improvement

Comments

@AntoLC
Copy link
Contributor

AntoLC commented Sep 4, 2023

Feature Request

List of the features needed in order to finish the transition from Grommet to Cunningham design system:

Tabs component

Modal component

Popover component


Textarea component


Select component

In this case we need to be able to add an image per option.
Marsha - select

@AntoLC AntoLC added feature request New feature or request improvement labels Sep 4, 2023
@AntoLC AntoLC changed the title ✨Select component with options type JSX.Element ✨Select needed feature in marsha Sep 4, 2023
@NathanVss
Copy link
Member

tbh all these requests seems legit to me. Are they sorted by priority?

@lebaudantoine
Copy link
Collaborator

@AntoLC , I'm curious if you might encounter a similar issue with the pop-in position as you did with the date picker component. Has its position already been made dynamic?

@NathanVss
Copy link
Member

@AntoLC , I'm curious if you might encounter a similar issue with the pop-in position as you did with the date picker component. Has its position already been made dynamic?

Can you give me more details about this please, I'm curious

@lebaudantoine
Copy link
Collaborator

yes ofc @NathanVss, please have a look to this merged PR here, initiated by @AntoLC

@AntoLC
Copy link
Contributor Author

AntoLC commented Sep 12, 2023

tbh all these requests seems legit to me. Are they sorted by priority?

Not sorted, no priority.

@AntoLC
Copy link
Contributor Author

AntoLC commented Sep 12, 2023

@AntoLC , I'm curious if you might encounter a similar issue with the pop-in position as you did with the date picker component. Has its position already been made dynamic?

Yes, I think there is the same problem. It doesn't seem to impact Marsha though.

@AntoLC
Copy link
Contributor Author

AntoLC commented Sep 15, 2023

Searcheable without filtering onFocus

With options = ["France", "England", "Spain"]
In a controlled way, if a value is set by default, let's say on France, when the user open the select, only France will be display in the options, it gives the feeling that only France option is available. It would be nice to have the full list onFocus, and to filter only when onInputChange.

@see https://mui.com/material-ui/react-autocomplete/#combo-box

@NathanVss
Copy link
Member

Searcheable without filtering onFocus

With options = ["France", "England", "Spain"] In a controlled way, if a value is set by default, let's say on France, when the user open the select, only France will be display in the options, it gives the feeling that only France option is available. It would be nice to have the full list onFocus, and to filter only when onInputChange.

@see https://mui.com/material-ui/react-autocomplete/#combo-box

100% agree, I was made this way because Downshift does it itself ( https://www.downshift-js.com/downshift#usage-with-getrootprops ) - I will change this, it's not intuitive +1

@NathanVss
Copy link
Member

Searcheable without filtering onFocus

With options = ["France", "England", "Spain"] In a controlled way, if a value is set by default, let's say on France, when the user open the select, only France will be display in the options, it gives the feeling that only France option is available. It would be nice to have the full list onFocus, and to filter only when onInputChange.

@see https://mui.com/material-ui/react-autocomplete/#combo-box

here is the related PR of this specific request ( #166 )

@AntoLC AntoLC changed the title ✨Select needed feature in marsha ✨Needed feature in marsha Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request improvement
Projects
None yet
Development

No branches or pull requests

3 participants