Bug 1454232 - Use toolbarbutton-attention-fill for syncing animation. r=dao draft
authorTim Nguyen <ntim.bugs@gmail.com>
Sun, 27 May 2018 20:32:19 +0200
changeset 800373 b69b8a789a96150fa4cf809f31b973df5cb9ab2b
parent 800364 bf36035bed13ab8fa1777955a299837d3e10d398
push id111330
push userbmo:ntim.bugs@gmail.com
push dateSun, 27 May 2018 18:37:33 +0000
reviewersdao
bugs1454232
milestone62.0a1
Bug 1454232 - Use toolbarbutton-attention-fill for syncing animation. r=dao MozReview-Commit-ID: 87V68Vud2ep
browser/themes/shared/customizableui/panelUI.inc.css
browser/themes/shared/toolbarbutton-icons.inc.css
--- a/browser/themes/shared/customizableui/panelUI.inc.css
+++ b/browser/themes/shared/customizableui/panelUI.inc.css
@@ -519,17 +519,17 @@ toolbarbutton[constrain-size="true"][cui
 @keyframes syncRotate {
   from { transform: rotate(0); }
   to { transform: rotate(360deg); }
 }
 
 #appMenu-fxa-icon[syncstatus="active"] > .toolbarbutton-icon,
 #PanelUI-remotetabs-syncnow[syncstatus="active"] > .toolbarbutton-icon {
   animation: syncRotate 0.8s linear infinite;
-  fill: #0a84ff;
+  fill: var(--toolbarbutton-icon-fill-attention);
 }
 
 #appMenu-fxa-status {
   -moz-box-align: center;
 }
 
 #appMenu-fxa-avatar {
   pointer-events: none;
--- a/browser/themes/shared/toolbarbutton-icons.inc.css
+++ b/browser/themes/shared/toolbarbutton-icons.inc.css
@@ -1,16 +1,17 @@
 :root {
   --toolbarbutton-icon-fill-attention: var(--lwt-toolbarbutton-icon-fill-attention, #0a84ff);
 }
 
 :root:-moz-lwtheme {
   --toolbarbutton-icon-fill-opacity: 1;
 }
 
+:root[lwt-popup-brighttext] panel,
 toolbar[brighttext] {
   --toolbarbutton-icon-fill-attention: var(--lwt-toolbarbutton-icon-fill-attention, #45a1ff);
 }
 
 .toolbarbutton-animatable-box,
 .toolbarbutton-1 {
   color: inherit;
   -moz-context-properties: fill, fill-opacity;