Skip to content

[9.0] Alerts table in Rule Preview panel fills container width (#214028)#214274

Merged
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-214028
Mar 12, 2025
Merged

[9.0] Alerts table in Rule Preview panel fills container width (#214028)#214274
kibanamachine merged 1 commit intoelastic:9.0from
kibanamachine:backport/9.0/pr-214028

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 9.0:

Questions ?

Please refer to the Backport tool documentation

@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 19:29
)

## Summary

Addresses elastic#213706.

I suspect that this stopped working when elastic#206523 removed the
`FullScreenContainer` from the inner StatefulEventsViewer component;
that change combined with the `display:flex` on this parent element
caused the table to no longer grow as expected.

Luckily, the fix here was simply not using flexbox on the container.
Since the table is the only element within this container, I saw no
reason why flexbox was needed here; testing has confirmed this.

### Screenshot (after fix; see bug for "before" image)
<kbd><img width="1709" alt="Screenshot 2025-03-11 at 5 35 43 PM"
src="https://github.com/user-attachments/assets/108be815-00e2-48b7-8821-e63566eefa09"
/></kbd>

(cherry picked from commit 01e31ea)
@kibanamachine kibanamachine force-pushed the backport/9.0/pr-214028 branch from aa1086f to 857c487 Compare March 12, 2025 19:30
@kibanamachine kibanamachine merged commit 17a2984 into elastic:9.0 Mar 12, 2025
9 checks passed
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #34 / dashboard app - group 4 dashboard listing page insights shows the insights panel and counts the views

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
securitySolution 8.8MB 8.8MB -27.0B

cc @rylnd

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