[EuiContextMenu] Fixed missing underlines on hover

- was broken for `<a>` links but not `<button>`s, doh
elastic · Nov 17, 2023 · 17c0dd7 · 17c0dd7
1 parent 0b799ff
commit 17c0dd7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/changelogs/upcoming/7373.md b/changelogs/upcoming/7373.md
@@ -4,6 +4,7 @@
 **Bug fixes**
 
 - Fixed non-`isPrimary` actions (collapsed within a popover) not showing the action description in a tooltip
+- Fixed missing underlines on `EuiContextMenu` link hover
 
 **Deprecations**
 

diff --git a/src/components/context_menu/context_menu_item.styles.ts b/src/components/context_menu/context_menu_item.styles.ts
@@ -27,13 +27,15 @@ export const euiContextMenuItemStyles = (euiThemeContext: UseEuiTheme) => {
       color: ${euiTheme.colors.text};
       outline-offset: -${euiTheme.focus.width};
 
-      &:enabled:hover,
-      &:enabled:focus {
-        text-decoration: underline;
-      }
+      &:not(:disabled) {
+        &:hover,
+        &:focus {
+          text-decoration: underline;
+        }
 
-      &:enabled:focus {
-        background-color: ${euiTheme.focus.backgroundColor};
+        &:focus {
+          background-color: ${euiTheme.focus.backgroundColor};
+        }
       }
     `,
     disabled: css`