-
-
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
[DropDownMenu] onChange is called twice #3121
Comments
Hi @vishalvijay . I tried reproducing the issue but I wasn't able to. Could you elaborate a bit more on this? Does clicking on the |
@vishalvijay Some hints on what to provide when submitting an issue: https://github.com/callemall/material-ui/blob/master/CONTRIBUTING.md#opening-an-issue |
Blind guess here: if this only happens on a mobile device then it could be due to onClick being fired for compat reasons after onTap is fired, causing a double press on the button. If that is the issue then the solution is likely the same as we did for double touch ripples: #1563 |
Closing this issue for now due to lack of response from the OP. I was able to work on something related to this and in mobiles it would be due to onTouch and onClick being fired together. The solution for double touch ripples should work in that case as pointed above. |
Clicking on a
MenuItem
inDropDownMenu
triggeronChange
handler twice.The text was updated successfully, but these errors were encountered: