Bug 1402311 - use fill-opacity instead of hardcoded fill colour for disabled overflow button in customize mode, r?mikedeboer draft
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Tue, 26 Sep 2017 15:17:26 +0100
changeset 671037 c10f071d72e8eef2f34213e7dedc117dc58efdab
parent 669742 5f3f19824efa14cc6db546baf59c54a0fc15ddc9
child 733391 3ece18b0834558aa6008e032e3a4aea7658ba1e2
push id81800
push usergijskruitbosch@gmail.com
push dateWed, 27 Sep 2017 10:12:02 +0000
reviewersmikedeboer
bugs1402311
milestone58.0a1
Bug 1402311 - use fill-opacity instead of hardcoded fill colour for disabled overflow button in customize mode, r?mikedeboer MozReview-Commit-ID: JKeA5ZNAh5J
browser/themes/shared/customizableui/customizeMode.inc.css
--- a/browser/themes/shared/customizableui/customizeMode.inc.css
+++ b/browser/themes/shared/customizableui/customizeMode.inc.css
@@ -441,25 +441,22 @@ toolbarpaletteitem[place=toolbar] > tool
 }
 
 #customization-panelWrapper > .panel-arrowbox {
   position: relative;
   height: 10px;
   margin-bottom: -1px;
 }
 
-/* The overflow button icon _looks_ disabled, but is also shown as [open]. */
-#nav-bar[customizing] > .overflow-button {
-  /* This color is the hard-coded #4c4c4c at 40% opacity as found in toolbarbutton-icons.inc.css */
-  fill: hsla(0,0%,30%,.4);
-}
-
+/* In customize mode, the overflow button should look both 'disabled' and 'open'.
+ * So we make the button fully opaque but fill the icon semi-transparent. */
 #nav-bar[customizing] > .overflow-button > .toolbarbutton-icon {
   background-color: var(--toolbarbutton-active-background);
   opacity: 1;
+  fill-opacity: 0.4;
 }
 
 #customization-panelWrapper > .panel-arrowbox > .panel-arrow[side="top"] {
 %ifdef XP_MACOSX
   list-style-image: var(--panel-arrow-image-vertical,
                         url("chrome://global/skin/arrow/panelarrow-vertical.png"));
   /* The OS X image is 2px narrower than the windows/linux one.
    * Add padding to compensate: */