Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix KibanaPageTemplate.BottomBar in serverless #168235

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

semd
Copy link
Contributor

@semd semd commented Oct 6, 2023

Summary

The latest changes applied here caused the KibanaPageTemplate.BottomBar to not work properly:

KibanaPageTemplate.BottomBar

The margin-left should be applied only to .euiBottomBar--fixed instead of all .euiBottomBar, --sticky and --static are already positioned correctly to the relative parent, so no margin needs to be applied.

@semd semd added release_note:skip Skip the PR/issue when compiling release notes v8.12.0 labels Oct 6, 2023
@semd semd requested a review from a team as a code owner October 6, 2023 14:11
@semd semd self-assigned this Oct 6, 2023
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

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 368.6KB 368.6KB +14.0B

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @semd

@semd semd enabled auto-merge (squash) October 6, 2023 15:59
Copy link
Contributor

@MichaelMarcialis MichaelMarcialis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code review only. LGTM.

@semd semd merged commit 0a2148f into elastic:main Oct 6, 2023
24 checks passed
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Oct 6, 2023
dej611 pushed a commit to dej611/kibana that referenced this pull request Oct 17, 2023
## Summary

The latest changes applied [here
](elastic#166840) caused the
`KibanaPageTemplate.BottomBar` to not work properly:


![KibanaPageTemplate.BottomBar](https://github.com/elastic/kibana/assets/17747913/5720fb62-1352-4140-b0f6-93c818689660)

The `margin-left` should be applied only to `.euiBottomBar--fixed`
instead of all `.euiBottomBar`, `--sticky` and `--static` are already
positioned correctly to the relative parent, so no margin needs to be
applied.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants