nav: Fix host switcher background split bug

cockpit-project · Dec 1, 2022 · 5682160 · 5682160
1 parent f259829
commit 5682160
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/pkg/shell/nav.scss b/pkg/shell/nav.scss
@@ -581,8 +581,8 @@ $desktop: $phone + 1px;
   }
 
   // Use the "current" background for actions color when item is selected
-  :where(.pf-m-current + .nav-item-actions) {
-    --pf-c-nav__link--BackgroundColor: var(--pf-c-nav--m-dark__item--m-current--BackgroundColor);
+  .pf-m-current + .nav-item-actions {
+    background-color: var(--pf-c-nav__link--m-current--BackgroundColor);
   }
 }
 
@@ -751,7 +751,7 @@ $desktop: $phone + 1px;
 
   // Redefine the background color for current links
   .area-ct-subnav .pf-c-nav__item {
-    > :is(.pf-m-current, .pf-m-current + .nav-item-actions) {
+    > .pf-m-current, .pf-m-current + .nav-item-actions {
       --pf-c-nav__link--m-current--BackgroundColor: var(--pf-global--BackgroundColor--dark-200);
     }
   }

diff --git a/test/reference b/test/reference
+ −		TestConnection-test_branding-login-screen-dark-pixels.png
+ −		TestHostSwitching-testBasic-edit-hosts-dark-pixels.png
+ −		TestHostSwitching-testBasic-hosts-sel-closed-dark-pixels.png
+ −		TestHostSwitching-testBasic-hosts-sel-closed-medium-pixels.png
+ −		TestHostSwitching-testBasic-hosts-sel-closed-mobile-pixels.png
+ −		TestHostSwitching-testBasic-hosts-sel-closed-pixels.png
+ −		TestHostSwitching-testBasic-nav-hosts-2-remotes-dark-pixels.png
+ −		TestHostSwitching-testBasic-nav-system-dark-pixels.png
+ −		TestPages-testMenuSearch-menu-search-dark-pixels.png
+ −		TestServices-testNotifyFailed-menu_error-dark-pixels.png
+ −		TestSuperuser-testBasic-topnav-privileged-dark-pixels.png
+ −		TestSuperuser-testBasic-topnav-unprivileged-dark-pixels.png