Skip to content

Conversation

@Dosant
Copy link
Contributor

@Dosant Dosant commented Dec 2, 2025

Summary

fix #242893

With the grid layout, the bottom bar could have overlapped the content area. This PR fixes it by adding padding to the content area equal to the height of the bottom bar, which pushes the content up. This is possible now due to elastic/eui#9211

Before:

Screenshot 2025-12-02 at 10 55 53

After:

Screenshot 2025-12-02 at 10 55 33

@Dosant Dosant requested review from a team as code owners December 2, 2025 09:56
@Dosant Dosant added release_note:skip Skip the PR/issue when compiling release notes backport:skip This PR does not require backporting Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// labels Dec 2, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@Dosant Dosant added the chrome-grid Work related to Chrome refactor to grid layout label Dec 2, 2025
@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #105 / visualize app - new charts library visualize Timelion visualization expression typeahead dynamic suggestions for argument values .es() should show field suggestions for timefield argument when index pattern set

Metrics [docs]

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 501.6KB 501.5KB -100.0B

@Dosant Dosant merged commit e9d50b6 into elastic:main Dec 2, 2025
23 checks passed
NicholasPeretti pushed a commit to NicholasPeretti/kibana that referenced this pull request Dec 2, 2025
## Summary

fix elastic#242893

With the grid layout, the bottom bar could have overlapped the content
area. This PR fixes it by adding padding to the content area equal to
the height of the bottom bar, which pushes the content up. This is
possible now due to elastic/eui#9211

### Before: 

<img width="1509" height="261" alt="Screenshot 2025-12-02 at 10 55 53"
src="https://github.com/user-attachments/assets/5edb3b98-9005-47b2-9c00-0fb9bc7fca51"
/>

### After:

<img width="1512" height="325" alt="Screenshot 2025-12-02 at 10 55 33"
src="https://github.com/user-attachments/assets/604a69a7-fa86-4787-b87e-313eb01607f0"
/>
JordanSh pushed a commit to JordanSh/kibana that referenced this pull request Dec 9, 2025
## Summary

fix elastic#242893

With the grid layout, the bottom bar could have overlapped the content
area. This PR fixes it by adding padding to the content area equal to
the height of the bottom bar, which pushes the content up. This is
possible now due to elastic/eui#9211

### Before: 

<img width="1509" height="261" alt="Screenshot 2025-12-02 at 10 55 53"
src="https://github.com/user-attachments/assets/5edb3b98-9005-47b2-9c00-0fb9bc7fca51"
/>

### After:

<img width="1512" height="325" alt="Screenshot 2025-12-02 at 10 55 33"
src="https://github.com/user-attachments/assets/604a69a7-fa86-4787-b87e-313eb01607f0"
/>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting chrome-grid Work related to Chrome refactor to grid layout release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Platform AppEx-SharedUX (formerly Global Experience) t// v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Chrome Grid] EuiBottomBar fixed causes page scroll

4 participants