-
Notifications
You must be signed in to change notification settings - Fork 693
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
[resize-observer-1] Turn ResizeObserver loop limit exceeded error into warning #5023
Comments
This error has eaten our entire quota of event reporting in sentry.io (bug report service) after a couple of days of integration. Throwing this as an error is causing financial cost and missing opportunities to fix real bugs. |
What are the next steps to move this forward? As more devs are starting using ResizeObserver, it makes them spend a lot of time figuring out what is going on until they find this issue and realize that their code is working fine. |
I left some thoughts here: #5488 (comment) This is overwhelming taking over the web dev. world and folks are mostly just ignoring it. Sometimes for legitimate reasons, sometimes not. #6173 attempts to describe ways to give levers as well so that developers can at least silence this on their own if they know it's right. |
Following up the discussion in WICG/resize-observer#38, they say that the error is benign, so it shouldn't trigger an error but a warning.
The text was updated successfully, but these errors were encountered: