Skip to content

[8.18] [Lens] fix drag and drop ghost after Chrome update to 134 (#213928)#214125

Merged
kibanamachine merged 3 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-213928
Mar 22, 2025
Merged

[8.18] [Lens] fix drag and drop ghost after Chrome update to 134 (#213928)#214125
kibanamachine merged 3 commits intoelastic:8.18from
kibanamachine:backport/8.18/pr-213928

Conversation

@kibanamachine
Copy link
Contributor

@kibanamachine kibanamachine commented Mar 12, 2025

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

…3928)

## Summary
Fixes elastic#213708

Fixes the ghost display. As @nickofthyme pointed out
[here](elastic#213708 (comment)),
the reason was using inset-inline-start: -10000px for the keyboard
handler that is one of the children of the draggable component. The
quick fix is to just move it outside of the html element that is
dragged. @markov00 please assesswhere it should be merged 🙏🏼

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 84ae9c9)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 12, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 12, 2025 11:37
mbondyra and others added 2 commits March 21, 2025 18:30
## Summary

Fixes elastic#213324

For some reason this PR elastic#213928
didn't catch that the changes make the test fail. This PR fixes it.
After this, we'll be able to backport the original PR.
(The pattern of using `data-attr-field` is already used in Discover so I
think it's a good opportunity to align)

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Nick Partridge <nicholas.partridge@elastic.co>
@nickofthyme nickofthyme self-assigned this Mar 21, 2025
@kibanamachine kibanamachine merged commit 4c972fd into elastic:8.18 Mar 22, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
discover 796.2KB 796.2KB +29.0B
eventAnnotationListing 213.8KB 213.8KB +29.0B
lens 1.5MB 1.5MB +60.0B
securitySolution 9.1MB 9.1MB +29.0B
slo 757.6KB 757.6KB +29.0B
total +176.0B

History

cc @mbondyra @nickofthyme

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants