Bug 1260697 - Use a class for menulist popups without icons. r=aceman a=rkent THUNDERBIRD451b1_2016042214_RELBRANCH
authorRichard Marti <richard.marti@gmail.com>
Wed, 30 Mar 2016 13:24:50 +0200
branchTHUNDERBIRD451b1_2016042214_RELBRANCH
changeset 26900 650acbe301c09e12fb50be0d7d3e810e3539669e
parent 26899 f4f878a88183d054db61cd3e90fe5cfb20ab5d76
child 26901 defe64a368bfde83c4cee1c544707491275e46a0
push id1850
push userclokep@gmail.com
push dateWed, 08 Mar 2017 19:29:12 +0000
treeherdercomm-esr52@028df196b2d9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersaceman, rkent
bugs1260697
Bug 1260697 - Use a class for menulist popups without icons. r=aceman a=rkent MozReview-Commit-ID: 6yOwW5Jl6ok
mail/base/content/extraCustomizeItems.xul
mail/themes/windows/mail/mailWindow1.css
--- a/mail/base/content/extraCustomizeItems.xul
+++ b/mail/base/content/extraCustomizeItems.xul
@@ -87,17 +87,19 @@
                  title="&mailViewsToolbarItem.title;"
                  align="center"
                  class="chromeclass-toolbar-additional">
       <label id="viewPickerLabel"
              value="&viewPicker.label;"
              control="viewPicker"
              accesskey="&viewPicker.accesskey;"/>
       <menulist id="viewPicker" oncommand="ViewChangeByMenuitem(event.target);">
-        <menupopup id="viewPickerPopup" onpopupshowing="RefreshViewPopup(this);">
+        <menupopup id="viewPickerPopup"
+                   class="noIconPopup"
+                   onpopupshowing="RefreshViewPopup(this);">
           <menuitem id="viewPickerAll" value="0"
                     label="&viewAll.label;"
                     type="radio"/>
           <menuitem id="viewPickerUnread" value="1"
                     label="&viewUnread.label;"
                     type="radio"/>
           <menuitem id="viewPickerNotDeleted" value="3"
                     label="&viewNotDeleted.label;"
--- a/mail/themes/windows/mail/mailWindow1.css
+++ b/mail/themes/windows/mail/mailWindow1.css
@@ -874,26 +874,26 @@ treechildren::-moz-tree-image(subjectCol
     -moz-appearance: none;
   }
 
   #folderLocationPopup menu,
   #folderLocationPopup menuitem {
     -moz-padding-start: 2px;
   }
 
+  .noIconPopup > menuseparator {
+    -moz-appearance: none;
+    margin-top: 3px;
+    margin-bottom: 3px;
+    padding-top: 0;
+    padding-bottom: 0;
+    border-top-color: #d7d7d7;
+  }
+
   @media (-moz-windows-default-theme) {
-    #afterViewPickerUnreadSeparator,
-    #afterViewPickerCustomViewsSeparator {
-      -moz-appearance: none;
-      margin: 3px 0;
-      padding-top: 0;
-      padding-bottom: 0;
-      border-top-color: ThreeDLightShadow;
-    }
-
     menulist {
       -moz-appearance: none;
       min-height: 24px;
       color: inherit;
       margin: 1px 4px;
       padding: 0 5px 1px !important;
       border: 1px solid;
       background: var(--toolbarbutton-hover-background);