-
Notifications
You must be signed in to change notification settings - Fork 83
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: hide scroller completely when dragging large grids #8351
Conversation
Quality Gate passedIssues Measures |
Hi @vursen and @web-padawan, when i performed cherry-pick to this commit to 23.5, i have encountered the following issue. Can you take a look and pick it manually? |
Description
The PR introduces a new solution to replace the previous workaround, which applied
min-height: 0
to the items container when dragging large grids. Instead, it now setsdisplay: none
on the scroller element. By completely hiding the grid content from the drag image, this change prevents crashes in Chromium browsers and improves performance in Safari. A similar change has also been applied to virtual-list.The related Chromium issue: https://issues.chromium.org/issues/383356871
Fixes #8235
Type of change