-
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
Solution nav add flex column styles #132252
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
|
||
display: flex; | ||
flex-direction: column; |
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.
Can you move this up to before the breakpoint mixin (line 13) otherwise it's hard to follow what styles are being applied directly to the selector.
💚 Build SucceededMetrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @semd |
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.
Double-checked the storybook in Chrome, FF and Safari for all breakpoints and different content types. Didn't seem to have any ill-effects.
Summary
This change is needed by #130023
It will be used in this PR #132210
The new Security Solution side nav needs to place some items at the bottom of the container, this change is needed to position them correctly. This style change is unnoticeable for the rest of the integrations.