Fix active dropdown item color (again).

aleksip · Feb 7, 2025 · 0b89667 · 0b89667
1 parent 577b61b
commit 0b89667
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 16 deletions.
diff --git a/themes/finna2/scss/finna/navigation.scss b/themes/finna2/scss/finna/navigation.scss
@@ -127,8 +127,8 @@
   }
   .dropdown-menu > li > .dropdown-item.active:not(:hover) {
     font-weight: bold;
-    --#{$prefix}dropdown-link-active-color: --#{$prefix}dropdown-link-color;
-    --#{$prefix}dropdown-link-active-bg: --#{$prefix}dropdown-link-bg;
+    color: var(--#{$prefix}dropdown-link-color);
+    background-color: var(--#{$prefix}dropdown-link-bg);
   }
 }
 

diff --git a/themes/finna2/scss/global/dropdowns-bootstrap.scss b/themes/finna2/scss/global/dropdowns-bootstrap.scss
@@ -32,21 +32,20 @@
   }
 }
 
-.dropdown-menu > li {
-  &:hover, &:active {
+.dropdown-menu > li > a {
+  display: block;
+
+  &:focus {
     background-color: inherit;
   }
-  > a {
-    display: block;
-
-    &:focus {
-      background-color: inherit;
-    }
-    &:hover,
-    &:active {
-      text-decoration: none;
-      color: $dropdown-link-hover-color;
-      background-color: $dropdown-link-hover-bg;
-    }
+  &.active:focus {
+    color: $dropdown-link-active-color;
+    background-color: $dropdown-link-active-bg;
+  }
+  &:hover,
+  &:active {
+    text-decoration: none;
+    color: $dropdown-link-hover-color;
+    background-color: $dropdown-link-hover-bg;
   }
 }