-
Notifications
You must be signed in to change notification settings - Fork 85
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
fix: use the most recent IntersectionObserver entry in form-layout #8596
Conversation
|
Just for reference: We are considering dropping IntersectionObserver completely in #8590. It seems to become unnecessary when FormLayout is based on CSS grid. However, the investigation is still ongoing. |
Hi @web-padawan and @web-padawan, when i performed cherry-pick to this commit to 24.6, i have encountered the following issue. Can you take a look and pick it manually? |
Hi @web-padawan and @web-padawan, when i performed cherry-pick to this commit to 24.5, i have encountered the following issue. Can you take a look and pick it manually? |
This ticket/PR has been released with Vaadin 24.7.0.alpha7 and is also targeting the upcoming stable 24.7.0 version. |
Description
Fixes #8564
I was unable to come up with a reliable test for this since the case where
IntersectionObserver
entries accumulate between callback invocations generally requires some slow rendering, like in the reproduction example provided.Type of change