Bug 1145372 - The add to reading list url bar button is larger than the other url bar buttons, r=markh. a=readinglist
authorFlorian Quèze <florian@queze.net>
Thu, 19 Mar 2015 15:50:23 -0700
changeset 248441 73a88d5a05d3ced5caf3b51c85773defe86eacc1
parent 248440 70a942a84ad58abe0963b800c186deb5a7fe8f6c
child 248442 7ed35febcb504b460ceed21b8e4aaa565150a9e3
push id7837
push userjwein@mozilla.com
push dateFri, 27 Mar 2015 00:27:16 +0000
treeherdermozilla-aurora@cb0db44ce60e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmarkh, readinglist
bugs1145372
milestone38.0a2
Bug 1145372 - The add to reading list url bar button is larger than the other url bar buttons, r=markh. a=readinglist
browser/themes/linux/browser.css
browser/themes/osx/browser.css
browser/themes/shared/readinglist/readinglist.inc.css
browser/themes/windows/browser.css
--- a/browser/themes/linux/browser.css
+++ b/browser/themes/linux/browser.css
@@ -1626,16 +1626,20 @@ richlistitem[type~="action"][actiontype=
 
 /* Popup blocker button */
 #page-report-button {
   list-style-image: url("chrome://browser/skin/Info.png");
 }
 
 %include ../shared/readinglist/readinglist.inc.css
 
+#readinglist-addremove-button {
+  padding: 0 2px;
+}
+
 /* Reader mode button */
 
 #reader-mode-button {
   -moz-appearance: none;
   padding: 0;
   list-style-image: url("chrome://browser/skin/reader-mode-16.png");
   -moz-image-region: rect(0, 16px, 16px, 0);
 }
--- a/browser/themes/osx/browser.css
+++ b/browser/themes/osx/browser.css
@@ -2520,16 +2520,22 @@ richlistitem[type~="action"][actiontype=
   #page-report-button:hover:active,
   #page-report-button[open="true"] {
     -moz-image-region: rect(0, 64px, 32px, 32px);
   }
 }
 
 %include ../shared/readinglist/readinglist.inc.css
 
+#readinglist-addremove-button {
+  padding: 3px;
+  -moz-padding-start: 2px;
+  -moz-padding-end: 1px;
+}
+
 /* Reader mode button */
 
 #reader-mode-button {
   -moz-appearance: none;
   padding: 0;
   list-style-image: url("chrome://browser/skin/reader-mode-16.png");
   -moz-image-region: rect(0, 16px, 16px, 0);
 }
--- a/browser/themes/shared/readinglist/readinglist.inc.css
+++ b/browser/themes/shared/readinglist/readinglist.inc.css
@@ -3,26 +3,25 @@
 #urlbar:not([focused]):not(:hover) #readinglist-addremove-button {
   display: none;
 }
 
 #readinglist-addremove-button {
   -moz-appearance: none;
   border: none;
   list-style-image: url("chrome://browser/skin/readinglist/icons.svg#addpage");
-  padding: 3px;
 }
 
 #readinglist-addremove-button:hover {
   border: none;
 }
 
 #readinglist-addremove-button > .toolbarbutton-icon {
-  width: 16px;
-  height: 16px
+  width: 14px;
+  height: 14px
 }
 
 #readinglist-addremove-button:hover {
   list-style-image: url("chrome://browser/skin/readinglist/icons.svg#addpage-hover");
 }
 
 #readinglist-addremove-button:active {
   list-style-image: url("chrome://browser/skin/readinglist/icons.svg#addpage-active");
--- a/browser/themes/windows/browser.css
+++ b/browser/themes/windows/browser.css
@@ -1573,16 +1573,20 @@ richlistitem[type~="action"][actiontype=
 
 #page-report-button:hover:active,
 #page-report-button[open="true"] {
   -moz-image-region: rect(0, 48px, 16px, 32px);
 }
 
 %include ../shared/readinglist/readinglist.inc.css
 
+#readinglist-addremove-button {
+  padding: 0 2px;
+}
+
 /* Reader mode button */
 
 #reader-mode-button {
   -moz-appearance: none;
   padding: 0;
   list-style-image: url("chrome://browser/skin/reader-mode-16.png");
   -moz-image-region: rect(0, 16px, 16px, 0);
 }