Skip to content
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.

Sign up for GitHub

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

Jump to bottom

"ResizeObserver loop limit exceeded" error with images #254

Closed
petyosi opened this issue Jan 8, 2021 · 10 comments
Closed

"ResizeObserver loop limit exceeded" error with images #254

petyosi opened this issue Jan 8, 2021 · 10 comments
Labels
bug Something isn't working released

Comments

@petyosi
Copy link
Owner

petyosi commented Jan 8, 2021

Discussion in this fork.

Repro here.

A possible fix is to wrap the callback call in RAF. Should test that, as it might cause render delays.
@petyosi petyosi added the bug Something isn't working label Jan 8, 2021
@petyosi petyosi closed this as completed in e0ad247 Jan 9, 2021
@github-actions
Copy link

github-actions bot commented Jan 9, 2021

🎉 This issue has been resolved in version 1.2.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@petyosi
Copy link
Owner Author

petyosi commented Jan 9, 2021

@edunad - please give this a try.

@edunad
Copy link

edunad commented Jan 11, 2021

Seems to be fixed, thanks!

@petyosi petyosi mentioned this issue Jan 23, 2021
Closed
petyosi added a commit that referenced this issue Jan 24, 2021
@petyosi
fix(): revert RAF for useSize
5475a10 
Fixes #269
Opens #254

A workaround is present here:
DevExpress/testcafe#4857 (comment)
@petyosi
Copy link
Owner Author

petyosi commented Jan 24, 2021

@edunad I had to revert that change - unfortunately, it introduced an issue with reverse scrolling (#269). Can you try the workaround from this discussion - DevExpress/testcafe#4857 (comment)

@hilts-vaughan hilts-vaughan mentioned this issue Apr 2, 2021
Open
@hilts-vaughan
Copy link

FYI This happens for legitimate reasons in that you are modifying the same thing you observe -- I think this is a legitimate use case and opened w3c/csswg-drafts#6173 to discuss it.

You can see the above for more info as well; please chime in if this is something that has caused you pain in the past.

@petyosi petyosi mentioned this issue May 26, 2021
Closed
@gabrielmicko
Copy link

I still have this issue in the newest version 2.2.1

@ychcg
Copy link

ychcg commented Oct 6, 2021 •
edited
Loading

I still have this issue in the newest version 2.2.1

I also have this issue after update version 2.2.0
I don't know it can help but I use scrollIntoView as new feature.

@seschwartz8
Copy link

I'm getting this error on 2.2.2

@petyosi
Copy link
Owner Author

petyosi commented Dec 20, 2021

See my previous comment for a possible workaround

@petyosi petyosi mentioned this issue Jun 29, 2022
Closed
@andrisole92

This comment was marked as spam.

Sign in to view
Repository owner locked and limited conversation to collaborators Jul 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working released
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@petyosi @hilts-vaughan @gabrielmicko @edunad @andrisole92 @seschwartz8 @ychcg