-
-
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
[material-ui][Dialog] Prevent onClick on the root element from being overwritten #41881
[material-ui][Dialog] Prevent onClick on the root element from being overwritten #41881
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wow, surprised we still have bugs of this class after so many years of battle testing.
Co-authored-by: Olivier Tassinari <[email protected]> Signed-off-by: Ryan Burr <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Left one comment.
Netlify deploy previewhttps://deploy-preview-41881--material-ui.netlify.app/ Bundle size reportDetails of bundle changes (Toolpad) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the pull request!
…overwritten (#41881) Signed-off-by: Ryan Burr <[email protected]> Co-authored-by: ZeeshanTamboli <[email protected]>
@ZeeshanTamboli thanks for the assist! |
This PR fixes #41417
onClick
prop from being overwritten by destructing and invoking on local handler.