Skip to content

[8.17] [Lens] Fix Firefox inline editor scroll (#228625)#228851

Merged
nickofthyme merged 4 commits intoelastic:8.17from
nickofthyme:backport/8.17/pr-228625
Jul 24, 2025
Merged

[8.17] [Lens] Fix Firefox inline editor scroll (#228625)#228851
nickofthyme merged 4 commits intoelastic:8.17from
nickofthyme:backport/8.17/pr-228625

Conversation

@nickofthyme
Copy link
Copy Markdown
Contributor

Backport

This will backport the following commits from main to 8.17:

Questions ?

Please refer to the Backport tool documentation

## Summary

Fixes an issues in Firefox where the Lens flyout configuration was not
scrollable.

https://github.com/user-attachments/assets/e8e57fc6-b2c8-4354-be17-d41cb42317df

Fixes elastic#227939

### Checklist

- [ ] Review the [backport
guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing)
and apply applicable `backport:*` labels.

Co-authored-by: Marta Bondyra <4283304+mbondyra@users.noreply.github.com>
(cherry picked from commit 90d35e0)

# Conflicts:
#	x-pack/platform/plugins/shared/lens/public/react_embeddable/mount.tsx
@nickofthyme nickofthyme added the backport This PR is a backport of another PR label Jul 21, 2025
@nickofthyme nickofthyme enabled auto-merge (squash) July 21, 2025 19:05
@nickofthyme nickofthyme requested a review from a team July 23, 2025 03:24
@elasticmachine
Copy link
Copy Markdown
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
lens 1.5MB 1.5MB -700.0B

History

@nickofthyme nickofthyme merged commit 44130c5 into elastic:8.17 Jul 24, 2025
8 checks passed
@nickofthyme nickofthyme deleted the backport/8.17/pr-228625 branch July 24, 2025 07:22
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.

3 participants