-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Lens] Improve Toolbar Responsiveness #130175
[Lens] Improve Toolbar Responsiveness #130175
Conversation
…ove-toolbar-responsiveness
Pinging @elastic/kibana-vis-editors @elastic/kibana-vis-editors-external (Team:VisEditors) |
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.
Way better. Thank you!
.../plugins/lens/public/editor_frame_service/editor_frame/workspace_panel/warnings_popover.scss
Show resolved
Hide resolved
...s/lens/public/editor_frame_service/editor_frame/workspace_panel/workspace_panel_wrapper.scss
Show resolved
Hide resolved
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.
This looks much better Michael, thank you! LGTM!
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.
LGTM
* reduce flex gutters, fix wrapping behavior, etc. * hide text at medium and lower bps for better fit * adjust full screen toolbar styles * expand hiding of button text to large bp * adjust flex item alignment and wrapping (cherry picked from commit d5d798f)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
* reduce flex gutters, fix wrapping behavior, etc. * hide text at medium and lower bps for better fit * adjust full screen toolbar styles * expand hiding of button text to large bp * adjust flex item alignment and wrapping (cherry picked from commit d5d798f) Co-authored-by: Michael Marcialis <[email protected]>
Summary
This PR fixes issues that cause the Lens toolbar to appear broken/odd at smaller viewport sizes. Notable changes include:
Closes issue #128428.
Screenshots of the changes at various viewport sizes
Checklist
Delete any items that are not applicable to this PR.