PENDING PR: hover effect on menus

cernbox · Apr 25, 2022 · 7992158 · 7992158
1 parent b31f1a2
commit 7992158
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 5 deletions.
diff --git a/src/components/atoms/OcDrop/OcDrop.vue b/src/components/atoms/OcDrop/OcDrop.vue
@@ -207,6 +207,26 @@ export default {
   .tippy-content {
     // note: needed so that the box shadow from `oc-box-shadow-medium` doesn't get suppressed
     padding: 8px;
+    li {
+      a,
+      button:not([role="switch"]) {
+        box-sizing: border-box;
+        padding: 6px;
+        &:focus,
+        &:hover {
+          background-color: var(--oc-color-background-hover);
+
+          text-decoration: none !important;
+          border-radius: 5px;
+        }
+        &:hover span {
+          color: var(--oc-color-swatch-brand-hover) !important;
+        }
+        span {
+          text-decoration: none !important;
+        }
+      }
+    }
   }
 }
 

diff --git a/src/components/molecules/OcBreadcrumb/OcBreadcrumb.vue b/src/components/molecules/OcBreadcrumb/OcBreadcrumb.vue
@@ -183,10 +183,6 @@ export default {
     #oc-breadcrumb-contextmenu-trigger > span {
       vertical-align: middle;
       border: 3px solid transparent;
-      &:hover {
-        background-color: var(--oc-color-background-hover);
-        border-radius: 5px;
-      }
     }
 
     #oc-breadcrumb-contextmenu li button {

diff --git a/src/components/molecules/OcTable/OcTable.vue b/src/components/molecules/OcTable/OcTable.vue
@@ -472,7 +472,7 @@ export default {
     transition: background-color $transition-duration-short ease-in-out;
   }
 
-  &-hover tr:hover span {
+  &-hover tr:hover td:not(:last-child) {
     color: var(--oc-color-swatch-brand-hover) !important;
   }