-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Lens] fix drag and drop overlay #162310
[Lens] fix drag and drop overlay #162310
Conversation
💚 Build Succeeded
Metrics [docs]Async chunks
To update your PR or re-run it, just comment with: |
Pinging @elastic/kibana-visualizations @elastic/kibana-visualizations-external (Team:Visualizations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally with Safari 👍
## Summary Fixes elastic#160735 When implementing drag and drop for the first time, to move smoothly between the main drop target and extra drop targets (duplicate or swap), I created an extra overlay path that keeps the elements visible when moving around (so when user drags out of the main drop target in the direction of extra targets, we don't loose their visibility) Turns out it started causing the 🔝 bug in the recent versions. This solution is not needed anymore though. Right now we keep elements visible for a few seconds when user drags out no matter where the mouse cursor is, so that's enough to go from main target to extra targets. Here's the overlay path I removed (I added yellow transparent background so it's more visible) <img width="485" alt="Screenshot 2023-07-20 at 13 34 41" src="https://github.com/elastic/kibana/assets/4283304/69acb1ff-1ada-4987-b47e-da282f8b992d">
Summary
Fixes #160735
When implementing drag and drop for the first time, to move smoothly between the main drop target and extra drop targets (duplicate or swap), I created an extra overlay path that keeps the elements visible when moving around (so when user drags out of the main drop target in the direction of extra targets, we don't loose their visibility)
Turns out it started causing the 🔝 bug in the recent versions.
This solution is not needed anymore though. Right now we keep elements visible for a few seconds when user drags out no matter where the mouse cursor is, so that's enough to go from main target to extra targets.
Here's the overlay path I removed (I added yellow transparent background so it's more visible)