ResizeObserver loop completed with undelivered notifications

[LookRain]

Describe the bug
ResizeObserver loop completed with undelivered notifications errors thrown on list scroll, especially for items with unspecified height

This happens both in Safari and Chrome

Safari: the errors are console.errored and clearly visible on the stories https://inokawa.github.io/virtua/?path=/story/basics-vlist--default

Chrome: it seems the errors are not printed by default, so you need to catch it on window

window.addEventListener('error', (event) => {
  console.error('captured error', event);
});

Or if you have extensions installed that captures uncaught errors, they might be printed as well (for example Metamask captures them but does not print the error details)

To Reproduce

1. just go to the stories and scroll on the React default example in Safari
2. https://github.com/LookRain/demo-resize-observer-error, which just copies the code of the default story and adds a error capture to make the errors visible in Chrome

Expected behavior
Probably should not throw all these errors? I'm not sure if it's a browser bug cause I see similar issues with react-virtuoso as well

Platform:

• OS: MacOS,
• Browser: Chrom, Safari
• Version of this package: latest
• Version of framework: react

Additional context
Add any other context about the problem here.

[inokawa]

https://github.com/inokawa/virtua?tab=readme-ov-file#what-is-resizeobserver-loop-completed-with-undelivered-notifications-error

Of course, it would be better if the error did not occur, but it's probably inevitable with current design.

[LookRain]

@inokawa sorry man i only searched thru the error in the issues, but forgot to do it in the freakin readme lol, apologies for the spam

[inokawa]

Reopen to try to fix.

[inokawa]

Probably it is caused by recursive rerendering with flushSync triggered by items resize, like:

items are measured -> rerender & range change -> new items are mounted -> items are measured -> rerender & range change -> ... -> error!

Some rough ideas to reduce errors:

• improve default size calculation logic on mount (less resize to less error)
• defer range change to after next rerender
• improve rerender logic on resize to remove flushSync