Bug 1394680 - Apply the white fill color to animatable boxes inside of toolbars that have bright text. We were already doing this for compact themes but missed doing it for lightweight themes. r?ntim draft
authorJared Wein <jwein@mozilla.com>
Thu, 07 Sep 2017 03:34:54 -0400
changeset 660593 d8cc102173b77d0c2cc94370ba38184783739b82
parent 660422 d8e238b811d3dc74515065ae8cab6c74baf0295f
child 660600 8c386102de2c11532562806f4c1ecb9a01ae845b
child 660814 3f7e5207a674d3832f5da9ae00cf99ecdd4619a7
push id78457
push userbmo:jaws@mozilla.com
push dateThu, 07 Sep 2017 07:35:13 +0000
reviewersntim
bugs1394680
milestone57.0a1
Bug 1394680 - Apply the white fill color to animatable boxes inside of toolbars that have bright text. We were already doing this for compact themes but missed doing it for lightweight themes. r?ntim MozReview-Commit-ID: 5uHES5JwFH0
browser/themes/shared/toolbarbutton-icons.inc.css
--- a/browser/themes/shared/toolbarbutton-icons.inc.css
+++ b/browser/themes/shared/toolbarbutton-icons.inc.css
@@ -8,16 +8,17 @@ toolbar[brighttext] {
 
 .toolbarbutton-animatable-box,
 .toolbarbutton-1 {
   -moz-context-properties: fill;
   fill: #4c4c4c;
 }
 
 .toolbarbutton-animatable-box[brighttext],
+toolbar[brighttext] .toolbarbutton-animatable-box,
 toolbar[brighttext] .toolbarbutton-1 {
   fill: #fff;
 }
 
 #back-button:-moz-locale-dir(rtl) > .toolbarbutton-icon,
 #forward-button:-moz-locale-dir(rtl) > .toolbarbutton-icon,
 #reload-button:-moz-locale-dir(rtl) > .toolbarbutton-icon,
 #nav-bar-overflow-button:-moz-locale-dir(rtl) > .toolbarbutton-icon,