quine-global-labs/mirror-cockpit-project-cockpit
1
0
Fork 0
mirror of https://github.com/cockpit-project/cockpit.git synced 2026-11-03 21:49:50 -09:00
Code Issues Projects Releases Packages Wiki Activity

shell: Don't style the System button label as a nav toggle on mobile

Browse source 
In the mobile layout the "System" label sits 4px lower than the
"Help" and "Session" labels and its box overflows the navbar.

The nav toggle rule also matched `#nav-system-item > .pf-v6-c-button__text`,
so the label span got its own `display: grid`, `grid-template-rows: 28px 1fr`
and `block-size: 100%`. It renders inside a 28px row of its own, which
pushes the text down. The button itself already matches
`button.ct-nav-toggle`, and the nested rules in that block target
menu-toggle/select children that the label span never has.

Dropping the selector puts the label at the same height as the other
navbar labels. Follow-up to #23083.

Signed-off-by: Peter Buchan Jr <pbuchan@redhat.com>
Assisted-by: Claude Opus 5.5
This commit is contained in:
Peter Buchan Jr 2026-10-16 21:49:20 -04:00 committed by Tomáš Matuš
parent 4bc7d15220
commit d3488cc1a3
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
pkg/shell/nav.scss
View file

@ -586,8 +586,7 @@ $desktop: $phone + 1px;
}
#host-toggle,
button.ct-nav-toggle,
#nav-system-item > .pf-v6-c-button__text {
button.ct-nav-toggle {
// Remove gap between text and icon
--pf-v6-c-button--Gap: 0;
--pf-v6-c-menu-toggle--Gap: 0;