Bug 1420287 - Add horizontal padding on top bar radio buttons in the Page Info dialog to match pre-58 style. r=bgrins a=lizzard
authorDão Gottwald <dao@mozilla.com>
Mon, 29 Jan 2018 17:02:38 +0100
changeset 452147 cd0aa60ce8d90555a8b801e99849ad61e07f43ca
parent 452146 368e7b7bea38c4ec30aeac600a2f4c0fe05e74dc
child 452148 f7c7cab7877b85a71e3c684e13b489c264026ffd
push id8629
push usercbrindusan@mozilla.com
push dateThu, 01 Feb 2018 10:54:28 +0000
treeherdermozilla-beta@cd0aa60ce8d9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbgrins, lizzard
bugs1420287
milestone59.0
Bug 1420287 - Add horizontal padding on top bar radio buttons in the Page Info dialog to match pre-58 style. r=bgrins a=lizzard MozReview-Commit-ID: AnoHSLmYE0K
browser/themes/linux/pageInfo.css
browser/themes/osx/pageInfo.css
browser/themes/windows/pageInfo.css
--- a/browser/themes/linux/pageInfo.css
+++ b/browser/themes/linux/pageInfo.css
@@ -16,16 +16,20 @@
   color: -moz-FieldText;
 }
 
 #viewGroup > radio[selected="true"] {
   background-color: Highlight;
   color: HighlightText;
 }
 
+#viewGroup > radio > .radio-label-box {
+  padding: 0 6px;
+}
+
 #topBar {
   border-bottom: 1px solid ThreeDShadow;
   padding-inline-start: 10px;
   background-color: -moz-Field;
   color: -moz-FieldText;
 }
 
 #generalTab {
--- a/browser/themes/osx/pageInfo.css
+++ b/browser/themes/osx/pageInfo.css
@@ -22,17 +22,17 @@
 #viewGroup > radio,
 #viewGroup > toolbarbutton {
   -moz-box-orient: vertical;
   -moz-box-align: center;
   -moz-appearance: toolbarbutton;
   font: menu;
   text-shadow: @loweredShadow@;
   margin: 0;
-  padding: 0 1px;
+  padding: 0 6px;
   height: 22px;
 }
 
 #viewGroup > radio[selected=true],
 #viewGroup > toolbarbutton[checked=true] {
   color: #FFF !important;
   text-shadow: rgba(0, 0, 0, 0.4) 0 1px;
 }
--- a/browser/themes/windows/pageInfo.css
+++ b/browser/themes/windows/pageInfo.css
@@ -7,31 +7,36 @@
 /* View buttons */
 #viewGroup {
   padding-inline-start: 10px;
 }
 
 #viewGroup > radio {
   list-style-image: url("chrome://browser/skin/pageInfo.png");
   -moz-appearance: none;
-  padding: 5px 3px 1px 3px;
-  margin: 0 1px;
   min-width: 4.5em;
+  margin: 0;
+  padding: 3px;
 }
 
 #viewGroup > radio:hover {
   background-color: #E0E8F6;
   color: black;
 }
 
 #viewGroup > radio[selected="true"] {
   background-color: #C1D2EE;
   color: black;
 }
 
+#viewGroup > radio > .radio-label-box {
+  margin: 0;
+  padding: 0 6px;
+}
+
 #topBar {
   border-bottom: 1px solid ThreeDLightShadow;
   padding-inline-start: 10px;
   background-color: -moz-Field;
   color: -moz-FieldText;
 }
 
 #generalTab {