Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

ResizeObserver loop limit exceeded

See original GitHub issue

Ever since that we started using react-cool-dimensions that we’ve been having a lot of logged errors:

ResizeObserver loop limit exceeded

Based on this StackOverflow answer it’s not really something to worry about, however, it can be improved by wrapping the ResizeObserver updates in a requestAnimationFrame.

Here’s an example of how a similar library (not a hook) solved this.

Do you think we can get such an improvement into react-cool-dimensions?

Issue Analytics

State:
Created 3 years ago
Reactions:1
Comments:24 (23 by maintainers)

Top GitHub Comments

2reactions

rfgamaralcommented, Mar 21, 2021

@rfgamaral I think this bug only leaves in old versions.

We have logs from the latest browser versions, so I don’t think this only happens on old versions.

Should we still need to care about it?

In my opinion, yes.

Everyone that uses this library with some logging service (like Sentry) will probably come across this “issue”. And I say “issue” because it’s not really affecting functionality (as far as we know), but we have around 34k events affecting 18k users on this “issue” alone. We could ignore it on our system, but based on the links I posted above, this should be easily fixable.

I understand that without reproducing it, it’s hard to understand if we are actually fixing anything. And I can’t reproduce it myself either, so the best I could do is try to deploy a version with a similar fix to the ones mentioned above, and see if the logs start to go away.

1reaction

rfgamaralcommented, Mar 23, 2021

Yes I did, @wellyshen, thanks for catching that 😅

Just updated our internal PR 😊