Bug 945539 - Use correct high-DPI styles for Metro download button [r=rsilveira]
authorMatt Brubeck <mbrubeck@mozilla.com>
Mon, 02 Dec 2013 17:39:14 -0800
changeset 173105 fc259acf93e343e8f611aad77c8fde7dfeb8c746
parent 173104 3fc2445c172691e544e0e563c9d18ef73d7d1dc7
child 173106 efd16375788e362ae5e48d7abcbeae005921ce7d
push id3224
push userlsblakk@mozilla.com
push dateTue, 04 Feb 2014 01:06:49 +0000
treeherdermozilla-beta@60c04d0987f1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersrsilveira
bugs945539
milestone28.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 945539 - Use correct high-DPI styles for Metro download button [r=rsilveira]
browser/metro/base/content/browser.xul
browser/metro/theme/browser.css
browser/metro/theme/platform.css
--- a/browser/metro/base/content/browser.xul
+++ b/browser/metro/base/content/browser.xul
@@ -255,17 +255,17 @@ Desktop browser's sync prefs.
             <toolbarbutton id="stop-button" class="urlbar-button"
                            command="cmd_stop"/>
           </hbox>
 
           <stack id="toolbar-contextual">
             <observes element="bcast_windowState" attribute="*"/>
             <observes element="bcast_urlbarState" attribute="*"/>
             <hbox id="toolbar-context-page" pack="end">
-              <circularprogressindicator id="download-progress"
+              <circularprogressindicator id="download-progress" class="appbar-primary"
                                      oncommand="MetroDownloadsView.onDownloadButton()"/>
               <toolbarbutton id="star-button" class="appbar-primary"
                              type="checkbox"
                              oncommand="Appbar.onStarButton()"/>
               <toolbarbutton id="pin-button" class="appbar-primary"
                              type="checkbox"
                              oncommand="Appbar.onPinButton()"/>
               <toolbarbutton id="menu-button" class="appbar-primary"
--- a/browser/metro/theme/browser.css
+++ b/browser/metro/theme/browser.css
@@ -736,26 +736,17 @@ documenttab[selected] .documenttab-selec
   visibility: collapse;
 }
 
 #download-progress:not([progress]) {
   visibility: collapse;
 }
 
 #download-progress {
-  width: 40px;
-  height: 40px;
   list-style-image: url(chrome://browser/skin/images/navbar-download.png);
-  -moz-image-region: rect(0px, 40px, 40px, 0px);
-}
-#download-progress:hover {
-  -moz-image-region: rect(0px, 80px, 40px, 40px);
-}
-#download-progress:active {
-  -moz-image-region: rect(0px, 120px, 40px, 80px);
 }
 
 #pin-button {
   list-style-image: url(chrome://browser/skin/images/navbar-pin.png);
 }
 
 #star-button {
   list-style-image: url(chrome://browser/skin/images/navbar-star.png);
--- a/browser/metro/theme/platform.css
+++ b/browser/metro/theme/platform.css
@@ -783,18 +783,18 @@ appbar toolbar[labelled] toolbarbutton {
 appbar toolbar[labelled] toolbarbutton > .toolbarbutton-text {
   display: block;
   padding-top: @metro_spacing_small@;
   font-size: 0.75rem;
 }
 
 /* Sprites */
 
-.appbar-primary > .toolbarbutton-icon,
-.appbar-secondary > .toolbarbutton-icon {
+.appbar-primary .toolbarbutton-icon,
+.appbar-secondary .toolbarbutton-icon {
   width: 40px;
   height: 40px;
 }
 
 /* Primary sprite format: one button per sprite.
 
    States from left to right:
    normal, hover, active/toggled, toggled+hover, toggled+active. */