-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Stateful sidenav] Fix side nav panel clipping #194268
[Stateful sidenav] Fix side nav panel clipping #194268
Conversation
Pinging @elastic/appex-sharedux (Team:SharedUX) |
packages/core/chrome/core-chrome-browser-internal/src/ui/project/navigation.tsx
Outdated
Show resolved
Hide resolved
Thanks for the review @Dosant ! Can you have another look? cheers 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Page load bundle
History
To update your PR or re-run it, just comment with: cc @sebelga |
Starting backport for target branches: 8.x https://github.com/elastic/kibana/actions/runs/11073552121 |
(cherry picked from commit c01de8c)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
# Backport This will backport the following commits from `main` to `8.x`: - [[Stateful sidenav] Fix side nav panel clipping (#194268)](#194268) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Sébastien Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2024-09-27T15:37:36Z","message":"[Stateful sidenav] Fix side nav panel clipping (#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","Feature:Chrome"],"title":"[Stateful sidenav] Fix side nav panel clipping","number":194268,"url":"https://github.com/elastic/kibana/pull/194268","mergeCommit":{"message":"[Stateful sidenav] Fix side nav panel clipping (#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194268","number":194268,"mergeCommit":{"message":"[Stateful sidenav] Fix side nav panel clipping (#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f"}}]}] BACKPORT--> Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
…elastic#194318) # Backport This will backport the following commits from `main` to `8.x`: - [[Stateful sidenav] Fix side nav panel clipping (elastic#194268)](elastic#194268) <!--- Backport version: 9.4.3 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Sébastien Loix","email":"sebastien.loix@elastic.co"},"sourceCommit":{"committedDate":"2024-09-27T15:37:36Z","message":"[Stateful sidenav] Fix side nav panel clipping (elastic#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:SharedUX","backport:prev-minor","Feature:Chrome"],"title":"[Stateful sidenav] Fix side nav panel clipping","number":194268,"url":"https://github.com/elastic/kibana/pull/194268","mergeCommit":{"message":"[Stateful sidenav] Fix side nav panel clipping (elastic#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194268","number":194268,"mergeCommit":{"message":"[Stateful sidenav] Fix side nav panel clipping (elastic#194268)","sha":"c01de8cf4db64d91c42f0170587f5f648c26fb3f"}}]}] BACKPORT--> Co-authored-by: Sébastien Loix <sebastien.loix@elastic.co>
In this PR I've fixed a bug in the side nav panel that was clipped when the side nav was collapsed.
In #191735 we allowed opening the panel from inside the stack management landing page. I didn't take into consideration that the side nav could be collapsed. This PR fixes it.
Screenshots
Before
After