-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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] Fix resize event leak #20272
Conversation
Oh wow, the switch happend in 02ba985#diff-d7f19b2c1ef03458dfa5ee43a1714637R367. |
Details of bundle changes.Comparing: 0e2e8ee...ee22be7 Details of page changes
|
I wonder how we could automate the catch of leaks like this one 🤔. How did you notice it? |
TypeScript would've caught this. |
Haha, I like this spirit 😁. |
Oh, it' been 5 months and still didn't reply 🤦♂️, sorry, I caught this by human eyes, while tracing the code. 👀, I don't remember what's was the issue that lead me to this one. |
No description provided.