-
-
Notifications
You must be signed in to change notification settings - Fork 32.4k
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
[Menu] Add prop to disable auto focus #11984
Conversation
@th317erd The concept sounds good to me. However, we need to find a different wording, it's conflicting with: |
@oliviertassinari That is exactly why I chose that prop name. I originally had it as |
@th317erd Let's say we can this pull request property |
@oliviertassinari Good thought! You are correct. I just pushed a new commit that renames this property. Thank you for your thoughts on this. :) |
@th317erd Thank you :) |
* Added Menu prop to disable auto focus * Update prop name per request of oliviertassinari * ready to be merged
I keep a fork of material ui simply and only because material ui Menu forcefully updates the element focus. I would greatly appreciate the merging of this PR to allow disabling this "feature". Thank you!