Bug 1349137 - Consolidate button color rules. r?johannh draft
authorDão Gottwald <dao@mozilla.com>
Tue, 21 Mar 2017 09:44:37 +0100
changeset 502114 ceb67bd284cc7c9b89f4efe6943565d618c8fbe7
parent 501879 5fe5dcf1c10a4523ba3f0a20295551462c2dae11
child 550064 435d0db379ea48e31081129477decfe178800254
push id50183
push userdgottwald@mozilla.com
push dateTue, 21 Mar 2017 08:46:00 +0000
reviewersjohannh
bugs1349137
milestone55.0a1
Bug 1349137 - Consolidate button color rules. r?johannh MozReview-Commit-ID: BS5WuQy7ykN
toolkit/themes/linux/global/button.css
--- a/toolkit/themes/linux/global/button.css
+++ b/toolkit/themes/linux/global/button.css
@@ -7,17 +7,17 @@
   ======================================================================= */
 
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
 /* :::::::::: button :::::::::: */
 
 button {
   -moz-appearance: button;
-  margin: 1px 5px 2px 5px;
+  margin: 1px 5px 2px;
   min-width: 6.3em;
   color: ButtonText;
   text-shadow: none;
 }
 
 .button-box {
   -moz-appearance: button-focus;
 }
@@ -28,32 +28,23 @@ button {
 
 .button-text {
   margin: 0 !important;
   text-align: center;
 }
 
 /* .......... hover state .......... */
 
-button:hover {
+button:hover:not(:-moz-any(:active,[disabled="true"],[open="true"],[checked="true"])) {
   color: -moz-buttonhovertext;
 }
 
-/* .......... active/open/checked state .......... */
-
-button:hover:active,
-button[open="true"],
-button[checked="true"] {
-  color: ButtonText;
-}
-
 /* .......... disabled state .......... */
 
-button[disabled="true"],
-button[disabled="true"]:hover:active {
+button[disabled="true"] {
   color: GrayText;
 }
 
 /* ::::: menu/menu-button buttons ::::: */
 
 button[type="menu-button"] {
   -moz-appearance: dualbutton;
 }