From 9183fd60975f901b978678cf4ee79a4555efcfdc Mon Sep 17 00:00:00 2001
From: Nicolas Chevobbe <nchevobbe@mozilla.com>
Date: Thu, 7 Nov 2024 16:01:43 +0000
Subject: [PATCH] Bug 1929594 - [devtools] Use --theme-text-color-inactive for
 disabled menu items instead of tweaking the opacity.
 r=devtools-reviewers,ochameau.

This makes the element nicely adapt to High Contrast Mode.

Differential Revision: https://phabricator.services.mozilla.com/D228191
---
 devtools/client/themes/tooltips.css | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/devtools/client/themes/tooltips.css b/devtools/client/themes/tooltips.css
index c0630394aefc..2b00d88abca2 100644
--- a/devtools/client/themes/tooltips.css
+++ b/devtools/client/themes/tooltips.css
@@ -579,7 +579,7 @@ strong {
     }
 
     &[disabled] {
-      opacity: 0.5;
+      color: var(--theme-text-color-inactive);
     }
 
     &[aria-checked="true"] {
@@ -613,10 +613,13 @@ strong {
   .menuitem > button.command:is([role="menuitem"],[role="menuitemcheckbox"],[role="link"]) {
     appearance: none;
     border: none;
-    color: var(--theme-arrowpanel-color);
     background-color: transparent;
     text-align: start;
     width: 100%;
+
+    &:not([disabled]) {
+      color: var(--theme-arrowpanel-color);
+    }
   }
 
   .checkbox-container .menuitem > .command > .label {