Bug 243078 Updates to fix some regressions from initial patch (bookmarks menu drag markers and Mac theme fix) a=asa

git-svn-id: svn://10.0.0.236/branches/MOZILLA_1_8_BRANCH@182091 18797224-902f-48f8-a5cc-f745e15eee43

mozilla/browser/themes/pinstripe/browser/browser.css

@@ -178,6 +178,10 @@ list-style-image: url("chrome://browser/skin/bookmarks/livemark-item.png");
   background: url("chrome://browser/skin/bookmarks/Toolbar-bookmarks.png") no-repeat;
 }
 
+#personal-bookmarks {
+  min-width: 21px; /* size of the chevron button */
+}
+
 .bookmarks-toolbar-customize {
   display: none;
   max-width: 15em !important;

mozilla/browser/themes/winstripe/browser/browser.css

@@ -97,6 +97,10 @@ toolbarbutton.bookmark-item[open="true"] {
   display: none;
 }
 
+#personal-bookmarks {
+  min-width: 21px; /* size of the chevron button */
+}
+
 #wrapper-personal-bookmarks .toolbarpaletteitem-box {
   width: 16px;
   height: 16px;
@@ -154,17 +158,17 @@ menuitem.bookmark-item {
 
 /* ..... drag and drop styles ..... */
 
-.bookmark-item[dragover-left="true"] {
-  -moz-border-left-colors: #000000;
+.bookmark-item[dragover-left="true"], .bookmark-item[dragover-left="true"] > * > .menu-drop-marker-box {
+  border-left: 1px solid #000000;
 }
-.bookmark-item[dragover-right="true"] {
-  -moz-border-right-colors: #000000;
+.bookmark-item[dragover-right="true"], .bookmark-item[dragover-right="true"] > * > .menu-drop-marker-box {
+  border-right: 1px solid #000000;
 }
-.bookmark-item[dragover-top="true"] {
-  -moz-border-top-colors: #000000;
+.bookmark-item[dragover-top="true"], .bookmark-item[dragover-top="true"] > * > .menu-drop-marker-box {
+  border-top: 1px solid #000000;
 }
-.bookmark-item[dragover-bottom="true"] {
-  -moz-border-bottom-colors: #000000;
+.bookmark-item[dragover-bottom="true"], .bookmark-item[dragover-bottom="true"] > * > .menu-drop-marker-box {
+  border-bottom: 1px solid #000000;
 }
 
 /* ::::: bookmark items ::::: */

mozilla/toolkit/content/widgets/menu.xml

@@ -33,14 +33,19 @@
 
   <binding id="menu" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:label class="menu-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
-      <xul:hbox anonid="accel">
-        <xul:label class="menu-accel" xbl:inherits="value=acceltext"/>
-      </xul:hbox>
-      <xul:hbox align="center" class="menu-right" chromedir="&locale.dir;"
-                xbl:inherits="_moz-menuactive,disabled">
-        <xul:image/>
-      </xul:hbox>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:label class="menu-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+          <xul:hbox anonid="accel">
+            <xul:label class="menu-accel" xbl:inherits="value=acceltext"/>
+          </xul:hbox>
+          <xul:hbox align="center" class="menu-right" chromedir="&locale.dir;"
+                    xbl:inherits="_moz-menuactive,disabled">
+            <xul:image/>
+          </xul:hbox>
+        </xul:box>
+      </xul:stack>
       <children includes="menupopup"/>
     </content>
     
@@ -52,10 +57,15 @@
 
   <binding id="menuitem" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:label class="menu-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
-      <xul:hbox anonid="accel">
-        <xul:label class="menu-accel" xbl:inherits="value=acceltext"/>
-      </xul:hbox>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:label class="menu-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+          <xul:hbox anonid="accel">
+            <xul:label class="menu-accel" xbl:inherits="value=acceltext"/>
+          </xul:hbox>
+        </xul:box>
+      </xul:stack>
     </content>
     
     <implementation>
@@ -68,66 +78,90 @@
 
   <binding id="menu-menubar" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:label class="menubar-text" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      <xul:box class="menu-inner-content">
+        <xul:label class="menubar-text" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      </xul:box>
       <children includes="menupopup"/>
     </content>
   </binding>
 
   <binding id="menu-menubar-iconic" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:image class="menubar-left" xbl:inherits="image=src"/>
-      <xul:label class="menubar-text" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      <xul:box class="menu-inner-content">
+        <xul:image class="menubar-left" xbl:inherits="image=src"/>
+        <xul:label class="menubar-text" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      </xul:box>
       <children includes="menupopup"/>
     </content>
   </binding>
 
   <binding id="menuitem-iconic" extends="chrome://global/content/bindings/menu.xml#menuitem">
     <content>
-      <xul:hbox class="menu-iconic-left" align="center" pack="center"
-                xbl:inherits="selected,_moz-menuactive,disabled,checked">
-        <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
-      </xul:hbox>
-      <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
-      <xul:hbox anonid="accel">
-        <xul:label class="menu-iconic-accel" xbl:inherits="value=acceltext"/>
-      </xul:hbox>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:hbox class="menu-iconic-left" align="center" pack="center"
+                    xbl:inherits="selected,_moz-menuactive,disabled,checked">
+            <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
+          </xul:hbox>
+          <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+          <xul:hbox anonid="accel">
+            <xul:label class="menu-iconic-accel" xbl:inherits="value=acceltext"/>
+          </xul:hbox>
+        </xul:box>
+      </xul:stack>
     </content>
   </binding>
 
   <binding id="menuitem-iconic-noaccel" extends="chrome://global/content/bindings/menu.xml#menuitem">
     <content>
-      <xul:hbox class="menu-iconic-left" align="center" pack="center"
-                xbl:inherits="selected,disabled,checked">
-        <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
-      </xul:hbox>
-      <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:hbox class="menu-iconic-left" align="center" pack="center"
+                    xbl:inherits="selected,disabled,checked">
+            <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
+          </xul:hbox>
+          <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+        </xul:box>
+      </xul:stack>
     </content>
   </binding>
 
   <binding id="menuitem-iconic-desc-noaccel" extends="chrome://global/content/bindings/menu.xml#menuitem">
     <content>
-      <xul:hbox class="menu-iconic-left" align="center" pack="center"
-                xbl:inherits="selected,disabled,checked">
-        <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
-      </xul:hbox>
-      <xul:label class="menu-iconic-text" xbl:inherits="value=label,accesskey,crop" crop="right" flex="1"/>
-      <xul:label class="menu-iconic-text menu-description" xbl:inherits="value=description" crop="right" flex="10000"/>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:hbox class="menu-iconic-left" align="center" pack="center"
+                    xbl:inherits="selected,disabled,checked">
+            <xul:image class="menu-iconic-icon" xbl:inherits="src=image,validate,src"/>
+          </xul:hbox>
+          <xul:label class="menu-iconic-text" xbl:inherits="value=label,accesskey,crop" crop="right" flex="1"/>
+          <xul:label class="menu-iconic-text menu-description" xbl:inherits="value=description" crop="right" flex="10000"/>
+        </xul:box>
+      </xul:stack>
     </content>
   </binding>
 
   <binding id="menu-iconic" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:hbox class="menu-iconic-left" align="center" pack="center">
-        <xul:image xbl:inherits="src=image"/>
-      </xul:hbox>
-      <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
-      <xul:hbox anonid="accel">
-        <xul:label class="menu-iconic-accel" xbl:inherits="value=acceltext"/>
-      </xul:hbox>
-      <xul:hbox class="menu-right" chromedir="&locale.dir;"
-                xbl:inherits="_moz-menuactive,disabled" align="center" pack="center">
-        <xul:image/>
-      </xul:hbox>
+      <xul:stack>
+        <xul:box class="menu-drop-marker-box"/>
+        <xul:box class="menu-inner-content">
+          <xul:hbox class="menu-iconic-left" align="center" pack="center">
+            <xul:image xbl:inherits="src=image"/>
+          </xul:hbox>
+          <xul:label class="menu-iconic-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+          <xul:hbox anonid="accel">
+            <xul:label class="menu-iconic-accel" xbl:inherits="value=acceltext"/>
+          </xul:hbox>
+          <xul:hbox class="menu-right" chromedir="&locale.dir;"
+                    xbl:inherits="_moz-menuactive,disabled" align="center" pack="center">
+            <xul:image/>
+          </xul:hbox>
+        </xul:box>
+      </xul:stack>
       <children includes="menupopup|template"/>
     </content>
     
@@ -139,7 +173,9 @@
   
   <binding id="menubutton-item" extends="chrome://global/content/bindings/menu.xml#menuitem-base">
     <content>
-      <xul:label class="menubutton-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      <xul:box class="menu-inner-content">
+        <xul:label class="menubutton-text" flex="1" xbl:inherits="value=label,accesskey,crop" crop="right"/>
+      </xul:box>
       <children includes="menupopup"/>
     </content>
   </binding>