-
Notifications
You must be signed in to change notification settings - Fork 6
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
ResizeObserver loop limit exceeded #328
Comments
Maybe this is something that could be solved internally by https://github.com/PrecisionNutrition/ember-resize-observer-service if we resolve #247? |
@elwayman02 I like your thoughts in #247 then having this issues addressed in that library. In the interim I created a PR that seems to work in our use case. I don't know if it is the right answer in all situations though... |
I think the best approach, if we "fix" this at all, is to simply catch the error and ignore it. |
Fixed by #506 |
There is a long ongoing discussion on this error, which can be safely ignored.
But since this error can happen and break tests, should we capture and ignore this error in this library to avoid having the consumers doing it themselves?
The text was updated successfully, but these errors were encountered: