Skip to content

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

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

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

Conversation

@kibanamachine
Copy link
Contributor

@kibanamachine kibanamachine commented Mar 12, 2025

Backport

This will backport the following commits from main to 8.16:

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:36
mbondyra and others added 2 commits March 21, 2025 18:26
## 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 cbe918e into elastic:8.16 Mar 22, 2025
8 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

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 788.4KB 788.4KB +29.0B
eventAnnotationListing 217.7KB 217.7KB +29.0B
lens 1.5MB 1.5MB +60.0B
securitySolution 8.5MB 8.5MB +29.0B
slo 756.1KB 756.1KB +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