Bug 1368679 - Increase hover opacity of window control buttons, apply the hover effect on the menu bar. r=dao draft
authorJohann Hofmann <jhofmann@mozilla.com>
Tue, 25 Jul 2017 17:26:37 +0200
changeset 615169 3c8fdc99caa20c96261e34d53e3389fc970accf0
parent 615105 07484bfdb96bc7297c404e377eea93f1d8ca4442
child 639092 63d27872522c51a1c92ecdcb5a608907894f65d9
push id70257
push userbmo:jhofmann@mozilla.com
push dateTue, 25 Jul 2017 15:29:26 +0000
reviewersdao
bugs1368679
milestone56.0a1
Bug 1368679 - Increase hover opacity of window control buttons, apply the hover effect on the menu bar. r=dao MozReview-Commit-ID: 2pw4UHOnzNE
browser/themes/windows/browser-aero.css
--- a/browser/themes/windows/browser-aero.css
+++ b/browser/themes/windows/browser-aero.css
@@ -185,22 +185,24 @@
         .titlebar-button > .toolbarbutton-icon {
           width: 10.8px;
           height: 10.8px;
         }
       }
 
       @media (-moz-windows-default-theme) {
 %ifdef MOZ_PHOTON_THEME
+        #main-menubar > menu[_moz-menuactive="true"],
         .titlebar-button:hover {
-          background-color: hsla(0,0%,100%,.12);
+          background-color: hsla(0,0%,100%,.22);
         }
         .titlebar-button:hover:active {
-          background-color: hsla(0,0%,100%,.22);
+          background-color: hsla(0,0%,100%,.32);
         }
+        #main-menubar > menu[_moz-menuactive="true"]:-moz-lwtheme-darktext,
         .titlebar-button:-moz-lwtheme-darktext:hover {
           background-color: hsla(0,0%,0%,.12);
         }
         .titlebar-button:-moz-lwtheme-darktext:hover:active {
           background-color: hsla(0,0%,0%,.22);
         }
 %else
         .titlebar-button:hover {
@@ -292,16 +294,22 @@
     /* Artificially draw window borders that are covered by lwtheme, see bug 591930.
      * Borders for win7 are below, win10 doesn't need them. */
     #main-window[sizemode="normal"] > #tab-view-deck > #browser-panel:-moz-lwtheme {
       border-top: 1px solid @toolbarShadowColor@;
     }
   }
 
   @media (-moz-windows-default-theme) {
+%ifdef MOZ_PHOTON_THEME
+    #main-menubar > menu {
+      -moz-appearance: none;
+    }
+%endif
+
     #main-menubar > menu:not(:-moz-lwtheme) {
       color: inherit;
     }
 
     /* Use a different color only on Windows 8 and higher for inactive windows.
      * On Win 7, the menubar fog disappears for inactive windows, and renders gray
      * illegible.
      */