-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Popper] Remove duplicate handleOpen call from effect #21106
Conversation
React.useEffect
with handleOpen
call
Could this surface an actual flaw in the usage of the ref? It feels like should be a state, not a ref, otherwise, why would we call handleOpen in? |
Details of bundle changes.Comparing: dbbf67f...ac75406 Details of page changes
|
Because it's trigger for us, that node is mounted – here is demo with what I tell about. PS: But maybe I didn't understand your question 🤔 |
@inomdzhon Thanks! Just lovely if you can remove code with confidence. |
Fix #21102