Skip to content

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

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

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

Conversation

@kibanamachine
Copy link
Contributor

@kibanamachine kibanamachine commented Mar 12, 2025

Backport

This will backport the following commits from main to 8.17:

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
@elasticmachine
Copy link
Contributor

elasticmachine commented Mar 12, 2025

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #55 / lens app - group 5 lens drag and drop tests keyboard drag and drop should drop a field to workspace
  • [job] [logs] FTR Configs #55 / lens app - group 5 lens drag and drop tests keyboard drag and drop should drop a field to workspace

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 773.2KB 773.2KB +29.0B
eventAnnotationListing 220.4KB 220.4KB +29.0B
lens 1.5MB 1.5MB +38.0B
securitySolution 8.8MB 8.8MB +29.0B
slo 738.2KB 738.2KB +29.0B
total +154.0B

History

cc @mbondyra

mbondyra and others added 2 commits March 21, 2025 18:28
## 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 530db06 into elastic:8.17 Mar 22, 2025
8 checks passed
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