Bug 590738 - Adjust color of menu separators in the app button menu. r=dao
authorKurt Schultz <supernova00@gmail.com>
Sat, 28 Aug 2010 11:24:43 -0400
changeset 51887 5fe0dbcf9d39f65a3fe1d4db53ab5fc1bc0a6345
parent 51886 dc2939f2640dc3d46cf1ee99fad45e171e100b4f
child 51888 134cbff3a612a66868ac891db54b049d66ec69f7
push idunknown
push userunknown
push dateunknown
reviewersdao
bugs590738
milestone2.0b6pre
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 590738 - Adjust color of menu separators in the app button menu. r=dao
browser/base/content/browser.xul
browser/themes/winstripe/browser/browser.css
--- a/browser/base/content/browser.xul
+++ b/browser/base/content/browser.xul
@@ -483,17 +483,17 @@
                </menu>
           </hbox>
           <menuitem id="appmenu_privateBrowsing"
                     class="menuitem-iconic"
                     label="&privateBrowsingCmd.start.label;"
                     startlabel="&privateBrowsingCmd.start.label;"
                     stoplabel="&privateBrowsingCmd.stop.label;"
                     command="Tools:PrivateBrowsing"/>
-          <menuseparator/>
+          <menuseparator class="appmenu-menuseparator"/>
           <hbox class="split-menuitem">
             <menuitem id="appmenu-edit-menuitem"
                       label="&editMenu.label;"
                       disabled="true"/>
             <toolbarbutton id="appmenu-cut"
                            class="appmenu-edit-button"
                            command="cmd_cut"
                            onclick="if (!this.disabled) hidePopup();"
@@ -507,17 +507,17 @@
                            class="appmenu-edit-button"
                            command="cmd_paste"
                            onclick="if (!this.disabled) hidePopup();"
                            tooltiptext="&pasteButton.tooltip;"/>
           </hbox>
           <menuitem id="appmenu_find"
                     label="&appMenuFind.label;"
                     command="cmd_find"/>
-          <menuseparator/>
+          <menuseparator class="appmenu-menuseparator"/>
           <menuitem id="appmenu_savePage"
                     label="&savePageCmd.label;"
                     command="Browser:SavePage"/>
           <menuitem id="appmenu_sendLink"
                     label="&sendPageCmd.label;"
                     command="Browser:SendLink"/>
           <hbox flex="1"
                 class="split-menuitem">
@@ -536,17 +536,17 @@
                           label="&printPreviewCmd.label;"
                           command="cmd_printPreview"/>
                 <menuitem id="appmenu_printSetup"
                           label="&printSetupCmd.label;"
                           command="cmd_pageSetup"/>
               </menupopup>
             </menu>
           </hbox>
-          <menuseparator/>
+          <menuseparator class="appmenu-menuseparator"/>
           <menu id="appmenu_developer"
                 label="&developerMenu.label;">
             <menupopup id="appmenu_developer_popup">
               <menuitem id="appmenu_webConsole"
                         label="&webConsoleCmd.label;"
                         type="checkbox"
                         oncommand="HUDConsoleUI.toggleHUD();"/>
               <menuitem id="appmenu_pageInspect"
@@ -558,17 +558,17 @@
                         label="&viewPageSourceCmd.label;"
                         command="View:PageSource"/>
               <menuseparator/>
               <menuitem label="&goOfflineCmd.label;"
                         type="checkbox"
                         oncommand="BrowserOffline.toggleOfflineStatus();"/>
             </menupopup>
           </menu>
-          <menuseparator/>
+          <menuseparator class="appmenu-menuseparator"/>
           <menuitem id="appmenu_fullScreen"
                     label="&fullScreenCmd.label;"
                     type="checkbox"
                     observes="View:FullScreen"/>
           <menuitem id="appmenu-quit"
                     class="menuitem-iconic"
 #ifdef XP_WIN
                     label="&quitApplicationCmdWin.label;"
--- a/browser/themes/winstripe/browser/browser.css
+++ b/browser/themes/winstripe/browser/browser.css
@@ -194,16 +194,27 @@ statusbarpanel#statusbar-display {
   #appmenuSecondaryPane {
     -moz-appearance: none;
     background-color: #f1f5fb;
     -moz-box-shadow: 1px 0 3px rgb(204,214,234) inset;
     border: 0;
     padding-top: 5px;
     font-family: "Segoe UI Semibold", "Segoe UI", sans-serif;
   }
+
+  .appmenu-menuseparator {
+    -moz-appearance: none;
+    margin-top: 3px;
+    margin-bottom: 3px;
+    -moz-margin-start: 30px;
+    padding: 0;
+    border-top: 1px solid #d6e5f5;
+    border-bottom: none;
+  }
+
   .appmenu-edit-button:not([disabled]):hover {
     border: 1px solid #b8d6fb;
     -moz-box-shadow: inset 0 0 1px white;
     background: -moz-linear-gradient(#fafbfd, #ebf3fd);
     -moz-transition: .2s ease-in;
   }
 }