Bug 1156913 - Use highlighttext color also for :active menus. r=Gijs, a=sledru
authorRichard Marti <richard.marti@gmail.com>
Wed, 22 Apr 2015 00:10:37 +0200
changeset 260254 98086516ce8f
parent 260253 309ed42a5999
child 260255 7c66212e4c09
push id727
push userryanvm@gmail.com
push date2015-04-23 14:45 +0000
treeherdermozilla-release@7c66212e4c09 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersGijs, sledru
bugs1156913
milestone38.0
Bug 1156913 - Use highlighttext color also for :active menus. r=Gijs, a=sledru
browser/themes/shared/customizableui/panelUIOverlay.inc.css
browser/themes/windows/customizableui/panelUIOverlay.css
--- a/browser/themes/shared/customizableui/panelUIOverlay.inc.css
+++ b/browser/themes/shared/customizableui/panelUIOverlay.inc.css
@@ -12,16 +12,17 @@
 % There's no calc() here (and therefore lots of calc() where this is used) because
 % we don't support nested calc(): https://bugzilla.mozilla.org/show_bug.cgi?id=968761
 %define menuPanelButtonWidth (@menuPanelWidth@ / 3 - 0.1px)
 %define exitSubviewGutterWidth 38px
 %define buttonStateHover :not(:-moz-any([disabled],[open],:active)):hover
 %define menuStateHover :not(:-moz-any([disabled],:active))[_moz-menuactive]
 %define buttonStateActive :not([disabled]):-moz-any([open],:hover:active)
 %define menuStateActive :not([disabled])[_moz-menuactive]:active
+%define menuStateMenuActive :not([disabled])[_moz-menuactive]
 
 %include ../browser.inc
 
 :root {
   --panel-ui-button-background-image: linear-gradient(to bottom, transparent, hsla(0,0%,100%,.3) 30%, hsla(0,0%,100%,.3) 70%, transparent),
                     linear-gradient(to bottom, transparent, hsla(210,54%,20%,.3) 30%, hsla(210,54%,20%,.3) 70%, transparent),
                     linear-gradient(to bottom, transparent, hsla(0,0%,100%,.3) 30%, hsla(0,0%,100%,.3) 70%, transparent);
   --panel-ui-button-background-size: 1px calc(100% - 1px), 1px calc(100% - 1px), 1px  calc(100% - 1px) !important;
--- a/browser/themes/windows/customizableui/panelUIOverlay.css
+++ b/browser/themes/windows/customizableui/panelUIOverlay.css
@@ -228,19 +228,19 @@ menu.subviewbutton > .menu-right:-moz-lo
     border: 1px solid ThreeDShadow;
   }
 
   .subviewbutton.panel-subview-footer,
   #BMB_bookmarksPopup .subviewbutton.panel-subview-footer {
     color: ButtonText;
   }
 
-  .subviewbutton@menuStateHover@,
+  .subviewbutton@menuStateMenuActive@,
   menuitem.panel-subview-footer@menuStateHover@,
   .subviewbutton.panel-subview-footer@buttonStateHover@,
   .subviewbutton.panel-subview-footer@buttonStateActive@,
   .subviewbutton@menuStateHover@ > .menu-accel-container,
   .PanelUI-subView .subviewbutton[shortcut]@buttonStateHover@::after,
-  #BMB_bookmarksPopup .panel-subview-footer@menuStateHover@ > .menu-text {
+  #BMB_bookmarksPopup .panel-subview-footer@menuStateMenuActive@ > .menu-text {
     background-color: Highlight;
     color: highlighttext !important;
   }
 }