Bug 1420287 - Add horizontal padding on top bar radio buttons in the Page Info dialog to match pre-58 style. r?bgrins draft
authorDão Gottwald <dao@mozilla.com>
Mon, 29 Jan 2018 17:02:38 +0100
changeset 748300 94159e7e93904097df05c910b08b8228468e87cf
parent 748299 4f7c34e223b6e68414f05f19b4843a1c48b12b4b
push id97122
push userdgottwald@mozilla.com
push dateMon, 29 Jan 2018 16:02:57 +0000
reviewersbgrins
bugs1420287
milestone60.0a1
Bug 1420287 - Add horizontal padding on top bar radio buttons in the Page Info dialog to match pre-58 style. r?bgrins 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 {