[ENHANCEMENT] Reduce navbars size #1492
Conversation
64a4724 to
7d48c7c
Compare
| '&': { | ||
| minHeight: '40px', | ||
| paddingLeft: 0, | ||
| paddingRight: 0.75, |
There was a problem hiding this comment.
| paddingRight: 0.75, | |
| paddingRight: 2, |
In order to have the same padding as "body" content
There was a problem hiding this comment.
Yeah ok it's a matter of either having the button zone aligned with the rest (padding 2)
or with the slider itself (padding 0.75)
Personally I have a slight preference for the 2nd one even it looks a bit like a hack here I agree :D.
There was a problem hiding this comment.
This hover effect size is pretty annoying, ok for me
|
I think, we could also remove the paddingTop of |
Signed-off-by: Antoine THEBAUD <antoine.thebaud@yahoo.fr>
7d48c7c to
f574553
Compare
|
by the way does it reduce the navbar size of the variable ? |
You mean the sticky bar I assume? Then yes, actually my bad I forgot to put a screenshot for this one, you can see one now in the PR description. |
|
Much much better 🥳 |
Description
Reduce the size of the various navbars (top one, the one with breadcrumb & timerange controls in read mode, the one with variables dropdowns & timerange controls in edit mode, also the sticky version of the latter) to spare vertical space for the dashboard content.
Screenshots
before/after comparison:
Checklist
[<catalog_entry>] <commit message>naming convention using one of the followingcatalog_entryvalues:FEATURE,ENHANCEMENT,BUGFIX,BREAKINGCHANGE,IGNORE.UI Changes