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
Popover does not listen to events unless it is open at the moment #9482
Conversation
The pull request has been inactive for a month. I'm closing. Maybe someone will review it later. |
How do I increase my chances of getting my PRs reviewed? |
@m2mathew Can you have a look at it? Thanks. |
I think it should be very simple, because the first thing the event handler does is "if we are not open, exit". So I think this should yield identical behavior to before, with the benefit of not spamming the browser with event handlers. |
@romanzenka Is this attached to an issue? Just want to make sure I am testing this accurately. 😄 |
Looks like this is related to #9481 |
#9481 yes, sorry. |
Looks great, @romanzenka. Thanks for digging up this bug and being patient with us! 💯 |
* call_em_all_-_master/master: fix: handle case where ref is null (mui#10006) The Popover does not listen to events unless it is open at the moment (mui#9482) [docs] Add v1 recommendation to home page (mui#9727) [Tabs] Add support for inline style override for parent container of InkBar (mui#9598) # Conflicts: # src/Popover/Popover.js
* commit 'fbfd478b85d02c1fc61ce525bee832f960792bfa': Use selectionEnd instead of selectionEnd to fix caret position issue (mui#10214) Static properties for reason string constants (mui#10300) conditionally apply overlay bkg (mui#9811) fix: handle case where ref is null (mui#10006) [EnhancedButton] Fix onClick event being fired twice on "Enter Key" press (mui#9439) The Popover does not listen to events unless it is open at the moment (mui#9482) [docs] Add v1 recommendation to home page (mui#9727) [Tabs] Add support for inline style override for parent container of InkBar (mui#9598) # Conflicts: # src/Popover/Popover.js # src/Slider/Slider.js # src/Snackbar/Snackbar.js
Attempt to fix Popovers listening to onScroll and onResize when they shouldn't.