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.

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 completed with undelivered notifications #189

Closed
igortas opened this issue Aug 29, 2023 · 4 comments
Closed

ResizeObserver loop completed with undelivered notifications #189

igortas opened this issue Aug 29, 2023 · 4 comments

Comments

@igortas
Copy link

igortas commented Aug 29, 2023

Hi,

I'm starting getting this issue for quite some time, even the library is updated frequently, currently happens also on the latest version.
In my scenario this error happens, when I frequently navigate between tabs which opens component per tab click (lazy loaded or not).

Part of wrapper code that I'm reusing on multiple places and providing some init config, nothing different from the basic example):

<PanelGroup
      direction='horizontal'
      disablePointerEventsDuringResize
    >
      {panes.map((child, index) => {
        return (
              <Panel
              >
                {child}
              <Panel>
                />
              )}
        );
      })}
    </PanelGroup>

Error:

ResizeObserver loop completed with undelivered notifications.
    at handleError (http://localhost:3000/static/js/bundle.js:269489:58)
    at http://localhost:3000/static/js/bundle.js:269508:7

Happens on microsoft edge, I'll check if happens the same on mozilla or not...

@bvaughn
Copy link
Owner

bvaughn commented Aug 30, 2023

I'm starting getting this issue for quite some time, even the library is updated frequently, currently happens also on the latest version.

That sounds odd, because ResizeObserver wasn't used in this project until ~2 weeks ago (#176) in the latest release.

I'm sorry to hear that you're hitting this. This isn't enough of a repro for me to investigate or fix though.

I do plan on removing ResizeObserver from panels that don't use pixel percentages in the next release (pending #182) which may help your case.

@igortas
Copy link
Author

igortas commented Aug 30, 2023

@bvaughn Found similar topics for this error on stackoverflow, and I've tried the weirdest fix, removing LastPass extension. So far looks good.

@bvaughn
Copy link
Owner

bvaughn commented Aug 30, 2023

That's interesting.

@bvaughn
Copy link
Owner

bvaughn commented Nov 12, 2023

#182 has been merged and I plan to release v56 this afternoon with the changes. I'm closing old issues in light of the rewrite because most of them are no longer relevant.

FWIW for this specific issue, ResizeObserver will only be referenced if pixel based constraints are used.

@bvaughn bvaughn closed this as completed Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants