Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Unhandled exception: ResizeObserver loop limit exceeded #1612

Closed
amacneil opened this issue Aug 9, 2021 · 5 comments · Fixed by #1698
Closed

Unhandled exception: ResizeObserver loop limit exceeded #1612

amacneil opened this issue Aug 9, 2021 · 5 comments · Fixed by #1698
Assignees
Labels
tech debt Technical cleanup with little or no user-facing impact

Comments

@amacneil
Copy link
Contributor

amacneil commented Aug 9, 2021

Description
We are getting thousands of these errors reported in Sentry.

As far as I understand this error doesn't have any user facing impact. If it's possible to prevent the error from happening, then we should do so. If not, we should skip reporting them via Sentry's ignoreErrors setting.

If we are going to ignore the error, we should do it client side rather than in Sentry UI, to avoid sending unnecessary data and also so that anyone configuring sentry to point at a different project will share the config.

@amacneil amacneil added the tech debt Technical cleanup with little or no user-facing impact label Aug 9, 2021
@jtbandes
Copy link
Member

jtbandes commented Aug 9, 2021

See also: WICG/resize-observer#38 (comment)

@defunctzombie
Copy link
Contributor

#1645

@jtbandes
Copy link
Member

nothing to see here

@amacneil
Copy link
Contributor Author

Still happening

@defunctzombie
Copy link
Contributor

I've suppressed it further for now but keeping this issue open until we figure out which parts of our resize observers cause this error. We should be able to configure them in a way to avoid this error. If we learn otherwise then we can close this out and leave the suppressions.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
tech debt Technical cleanup with little or no user-facing impact
Development

Successfully merging a pull request may close this issue.

3 participants