Skip to content

[Actionable Obs] Remove z-index override from Synthetics waterfall flyout#246343

Merged
fkanout merged 5 commits intoelastic:mainfrom
fkanout:remove-z-index-override
Dec 16, 2025
Merged

[Actionable Obs] Remove z-index override from Synthetics waterfall flyout#246343
fkanout merged 5 commits intoelastic:mainfrom
fkanout:remove-z-index-override

Conversation

@fkanout
Copy link
Copy Markdown
Contributor

@fkanout fkanout commented Dec 15, 2025

Summary

Related to #245413

Removes the manual z-index: euiZLevel5 override from the waterfall flyout component to ensure compatibility with the new EUI Flyout System. The override was lowering the flyout's z-index below EUI's default.

This override is very old (2021) PR, and I couldn't identify why. Keeping or removing that override didn't change anything in the UI.

@fkanout fkanout self-assigned this Dec 15, 2025
@fkanout fkanout requested a review from a team as a code owner December 15, 2025 11:46
@fkanout fkanout added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. v9.3.0 labels Dec 15, 2025
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/actionable-obs-team (Team:actionable-obs)

@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@github-actions github-actions bot added the author:actionable-obs PRs authored by the actionable obs team label Dec 15, 2025
@fkanout fkanout changed the title [Actionable Obs] Remove z-index override [Actionable Obs] Remove z-index override from Synthetics waterfall flyout Dec 15, 2025
@fkanout fkanout enabled auto-merge (squash) December 15, 2025 13:43
@fkanout fkanout merged commit 345d78d into elastic:main Dec 16, 2025
13 checks passed
@elasticmachine
Copy link
Copy Markdown
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
synthetics 1.0MB 1.0MB -36.0B

History

cc @fkanout

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

Labels

author:actionable-obs PRs authored by the actionable obs team backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. Team:obs-ux-management v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants