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] fix warning on the text-field component page #1028

Merged
merged 1 commit into from
Jul 4, 2015
Merged

[DropDownMenu] fix warning on the text-field component page #1028

merged 1 commit into from
Jul 4, 2015

Conversation

oliviertassinari
Copy link
Member

The warning is Warning: Failed propType: Invalid prop selectedIndex of type string supplied to Menu, expected number. Check the render method of DropDownMenu..
Since the selectedIndex was a string, in some cases, the selected prop wasn't set correctly.

hai-cea pushed a commit that referenced this pull request Jul 4, 2015
[DropDownMenu] fix warning on the text-field component page
@hai-cea hai-cea merged commit 8d2aae6 into mui:master Jul 4, 2015
@hai-cea
Copy link
Member

hai-cea commented Jul 4, 2015

Thanks @oliviertassinari

@oliviertassinari oliviertassinari deleted the drop-down-menu branch August 14, 2015 08:33
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Nov 10, 2020
* Update packages

* Fix rollup errors, enable umd bundle

* Remove unused deps from lib/package.json

* Use rollup-plugin-terser instead of uglify
@zannager zannager added component: menu This is the name of the generic UI component, not the React module! component: text field This is the name of the generic UI component, not the React module! labels Mar 14, 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! component: text field This is the name of the generic UI component, not the React module!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants