Bug 1316722 - [e10s] <select> dropdown list should handle Full/Text Zoom on Windows. r=mconley draft
authorJared Wein <jwein@mozilla.com>
Fri, 18 Nov 2016 13:11:32 -0500
changeset 441318 e9b20b0d01afeec91ed2e1b597d0e2f82eeec435
parent 441122 46127b3a981bceb0413c8199849f4e47afc949da
child 537526 2b33bf6e99322d05be94407195e17220ee2183eb
push id36391
push userjwein@mozilla.com
push dateFri, 18 Nov 2016 18:13:33 +0000
reviewersmconley
bugs1316722
milestone53.0a1
Bug 1316722 - [e10s] <select> dropdown list should handle Full/Text Zoom on Windows. r=mconley MozReview-Commit-ID: 1QVeHfsBDlM
browser/themes/windows/browser.css
--- a/browser/themes/windows/browser.css
+++ b/browser/themes/windows/browser.css
@@ -2663,29 +2663,26 @@ notification.pluginVulnerable > .notific
   -moz-border-top-colors: GrayText;
   -moz-border-right-colors: GrayText;
   -moz-border-bottom-colors: GrayText;
   -moz-border-left-colors: GrayText;
 }
 
 #ContentSelectDropdown > menupopup > menucaption,
 #ContentSelectDropdown > menupopup > menuitem {
-  padding: 4px 6px;
+  padding: 0 6px;
+  border-width: 0;
 }
 
 #ContentSelectDropdown > menupopup > menucaption > .menu-iconic-text,
 #ContentSelectDropdown > menupopup > menuitem > .menu-iconic-text {
-  font: message-box;
-  font-size: 11px;
-  /**
-   * Remove the extra vertical padding set by menu.css since
-   * the menuitem itself will include enough padding.
-   */
-  padding-top: 0px;
-  padding-bottom: 0px;
+  /* Padding should follow the 4/12 ratio, where 12px is the default font-size
+     with 4px being the preferred padding size. */
+  padding-top: .3333em;
+  padding-bottom: .3333em;
 }
 
 #ContentSelectDropdown > menupopup > menucaption > .menu-iconic-text {
   font-weight: bold;
 }
 
 #ContentSelectDropdown > menupopup > menuitem[_moz-menuactive="true"][disabled="true"] {
   color: GrayText;
@@ -2695,13 +2692,15 @@ notification.pluginVulnerable > .notific
 #ContentSelectDropdown > menupopup > menucaption {
   background-color: buttonface;
 }
 
 #ContentSelectDropdown > menupopup > menucaption[disabled="true"] {
   color: GrayText;
 }
 
-#ContentSelectDropdown > .isOpenedViaTouch > menucaption,
-#ContentSelectDropdown > .isOpenedViaTouch > menuitem {
-  padding-top: 11px;
-  padding-bottom: 11px;
+#ContentSelectDropdown > .isOpenedViaTouch > menucaption > .menu-iconic-text,
+#ContentSelectDropdown > .isOpenedViaTouch > menuitem > .menu-iconic-text {
+  /* Touch padding should follow the 11/12 ratio, where 12px is the default
+     font-size with 11px being the preferred padding size. */
+  padding-top: .9167em;
+  padding-bottom: .9167em;
 }