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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Avoid ResizeObserver loop limit exceeded #335
Conversation
nice 馃檶 LGTM 馃攣 were we seeing this on dashboards? is it specific to browser versions? (I haven't seen it before, or haven't been aware of seeing it :) |
@williaster @michellethomas found it while working on integration test in Cypress. It crashes the test on dashboard page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the PR @kristw!
published in |
fix(ngrid): Avoid ResizeObserver loop limit exceeded See airbnb/visx#335
Copies the approach used in airbnb/visx#335 to fix a "ResizeObserver loop limit exceeded" error
Copies the approach used in airbnb/visx#335 to fix a "ResizeObserver loop limit exceeded" error
Copies the approach used in airbnb/visx#335 to fix a "ResizeObserver loop limit exceeded" error # Conflicts: # src/Utils.tsx
Copies the approach used in airbnb/visx#335 to fix a "ResizeObserver loop limit exceeded" error
Copies the approach used in airbnb/visx#335 to fix a "ResizeObserver loop limit exceeded" error
馃悰 Bug Fix
ResizeObserver
loop limit exceeded. The issues surfaces on Chrome version >=64.Reference: WICG/resize-observer#38
Solution was to wrap the resize call in
requestAnimationFrame
to avoid infinite loopReference:
cc: @michellethomas