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

[Bug]: Tooltip (auto align): ResizeObserver loop completed with undelivered notifications #17461

Open
2 tasks done
JoshDKing opened this issue Sep 17, 2024 · 1 comment
Open
2 tasks done
Labels
severity: 2 https://ibm.biz/carbon-severity type: bug 🐛

Comments

@JoshDKing
Copy link

JoshDKing commented Sep 17, 2024

Package

@carbon/react

Browser

Chrome

Package version

v127.0.6533.120

React version

v^17.0.2

Description

When using a Popover/Tooltip and the auto align prop there are rare instances where there is an issue with the resize observer:

ERROR
ResizeObserver loop completed with undelivered notifications.

Its important to note we are using auto align in many places with NO ISSUE, its just on rare occasions when paired with the resize observer. If I were to change these from using auto align to lets say top the error goes away.

We are currently using "use-resize-observer": "^6.1.0",
I was able to resolve the issue by upgrading from ^6.1.0 -> ^8.0.0. However might be best to go ahead and upgrade it to the latest version 9.1.0.

Thought it could be worth updating the dependancy of use-resize-observer to 9.1.0

This is pretty difficult to reproduce unfortunately.

Link to SLACK discussion (IBM INTERAL)

Reproduction/example

Note able to do in sandbox unfortunately

Steps to reproduce

This is pretty difficult to reproduce unfortunately.

Suggested Severity

Severity 3 = User can complete task, and/or has a workaround within the user experience of a given component.

Application/PAL

No response

Code of Conduct

@tay1orjones
Copy link
Member

Thanks for opening this! Despite the lack of reproduction I still think it makes sense to update the dependency, especially since you've confirmed this fixes it for you locally.

I'm not sure if our usage hits any of the breaking changes of the new versions. I know we can remove the polyfill because we already have an issue for it, and they've removed it from that package in the latest major.

@tay1orjones tay1orjones added severity: 3 https://ibm.biz/carbon-severity severity: 2 https://ibm.biz/carbon-severity and removed status: needs triage 🕵️‍♀️ severity: 3 https://ibm.biz/carbon-severity labels Sep 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity: 2 https://ibm.biz/carbon-severity type: bug 🐛
Projects
Status: ⏱ Backlog
Development

No branches or pull requests

2 participants