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

DropDownMenu inconsistency #580

Closed
afhammad opened this issue Apr 25, 2015 · 3 comments
Closed

DropDownMenu inconsistency #580

afhammad opened this issue Apr 25, 2015 · 3 comments
Labels
component: menu This is the name of the generic UI component, not the React module!

Comments

@afhammad
Copy link

the DropDownMenu onChange behaves different to the normal select input onChange in that you cannot obtain the value from the event object but rather have to accept additional arguments. This breaks any form helpers relying on event.target.value and requires application code to be more tightly coupled with Material UI.

Also the lack of keyboard support (ex: jump to letter when a key is typed, or selecting an item using space/enter) makes it inferior to the default select input.

I'd be happy to contribute if you see these as flaws

@lananelson
Copy link

+1

@hai-cea
Copy link
Member

hai-cea commented Jun 8, 2015

Thanks guys - totally agree with this. Please check out discussions on #51

@hai-cea hai-cea closed this as completed Jun 8, 2015
siriwatknp added a commit to siriwatknp/material-ui that referenced this issue Jun 16, 2021
@oliviertassinari oliviertassinari added the status: waiting for maintainer These issues haven't been looked at yet by a maintainer label Dec 25, 2022
@zannager zannager added component: menu This is the name of the generic UI component, not the React module! and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: menu This is the name of the generic UI component, not the React module!
Projects
None yet
Development

No branches or pull requests

6 participants
@afhammad @hai-cea @lananelson @oliviertassinari @zannager and others