Bug 1343570 - menupopup appearance for customoptionstyling needs to be in popup.css instead of menu.css because menu.css doesn't have access to the parent popup. draft
authorJared Wein <jwein@mozilla.com>
Thu, 02 Mar 2017 15:14:04 -0500
changeset 492098 e43978323d06bdbc30d07ec5599b39982953b7c6
parent 492009 d29f84406483c721a13cf9a52936ecced0c5c98a
child 547649 7181409e454206155ef116ea55e59cd11be3aedc
push id47529
push userbmo:jaws@mozilla.com
push dateThu, 02 Mar 2017 20:14:36 +0000
bugs1343570
milestone54.0a1
Bug 1343570 - menupopup appearance for customoptionstyling needs to be in popup.css instead of menu.css because menu.css doesn't have access to the parent popup. MozReview-Commit-ID: LWkcjsN0mFt
toolkit/themes/osx/global/menu.css
toolkit/themes/osx/global/popup.css
--- a/toolkit/themes/osx/global/menu.css
+++ b/toolkit/themes/osx/global/menu.css
@@ -130,17 +130,16 @@ menupopup > menucaption {
 }
 
 menu[_moz-menuactive="true"],
 menuitem[_moz-menuactive="true"] {
   color: -moz-mac-menutextselect;
   background-color: Highlight;
 }
 
-menupopup[customoptionstyling="true"],
 menuitem[customoptionstyling="true"] {
   -moz-appearance: none;
   padding-top: 0;
   padding-bottom: 0;
 }
 
 /* ::::: menu/menuitems in menulist popups ::::: */
 
--- a/toolkit/themes/osx/global/popup.css
+++ b/toolkit/themes/osx/global/popup.css
@@ -134,8 +134,14 @@ menulist[editable="true"] > menupopup {
 
 menulist > menupopup > .popup-internal-box {
   padding: 0;
 }
 
 menulist:not([editable="true"]) > menupopup {
   padding: 4px 0;
 }
+
+menupopup[customoptionstyling="true"] {
+  -moz-appearance: none;
+  padding-top: 0;
+  padding-bottom: 0;
+}