Bug 382893 - "Purple "I" separator that appears on bookmark toolbar is ugly" [p=dao@mozilla.com (Dão Gottwald) ui-r=beltzner r=Mano a=blocking-firefox3+]

browser/components/places/content/toolbar.xml

@@ -838,23 +838,26 @@
             if (!ib.hasAttribute("dragging"))
               ib.setAttribute("dragging", "true");
             var ind = ib.firstChild;
+            var halfInd = ind.boxObject.width / 2;
             var direction = document.defaultView.getComputedStyle(this._self.parentNode, "").direction;
             if (direction == "ltr") {
+              halfInd = Math.ceil(halfInd);
               if (!this._self.childNodes.length)
-                ind.style.marginLeft = 0 - this._self.boxObject.x - 7 + 'px'
+                ind.style.marginLeft = 0 - this._self.boxObject.x - halfInd + 'px'
               else if (dropPoint.beforeIndex == -1)
                 ind.style.marginLeft = this._self.lastChild.boxObject.x + 
-                                       this._self.lastChild.boxObject.width - this._self.boxObject.x - 7 + 'px';
+                                       this._self.lastChild.boxObject.width - this._self.boxObject.x - halfInd + 'px';
               else
                 ind.style.marginLeft = this._self.childNodes[dropPoint.beforeIndex].boxObject.x -
-                                       this._self.boxObject.x - 7 + 'px';
+                                       this._self.boxObject.x - halfInd + 'px';
             } else {
+              halfInd = Math.floor(halfInd);
               if (dropPoint.beforeIndex == -1 || !this._self.childNodes.length)
                 ind.style.marginRight = '0px';
               else
                 ind.style.marginRight = (this._self.childNodes[this._self.childNodes.length - 1].boxObject.x +
                                          this._self.childNodes[this._self.childNodes.length - 1].boxObject.width) -
-                                        (this._self.childNodes[dropPoint.beforeIndex].boxObject.x) - 5 + 'px';
+                                        (this._self.childNodes[dropPoint.beforeIndex].boxObject.x) - halfInd + 'px';
             }
             // Clear out old folder information
             this._clearOverFolder();

browser/themes/gnomestripe/browser/browser.css

@@ -179,18 +179,17 @@ menuitem.bookmark-item {
 }
 
 .toolbar-drop-indicator {
-  height: 16px;
-  width: 5px;
-  margin-bottom: -8px;
+  width: 9px;
+  height: 18px;
+  margin-bottom: -6px;
   position: relative;
   background: url("chrome://browser/skin/places/toolbarDropMarker.png") 50% 50% no-repeat;
 }
 
 .toolbar-drop-indicator-bar {
   visibility: hidden;
-  height: 16px;
-  margin-bottom: -16px;
-  margin-left: 4px;
+  height: 18px;
+  margin-bottom: -18px;
   position: relative;
 }
 

browser/themes/pinstripe/browser/browser.css

@@ -1561,18 +1561,17 @@ tabpanels.plain {
 
 /* Personal toolbar */
 .toolbar-drop-indicator {
-  height: 16px;
-  width: 5px;
-  margin-bottom: -8px;
+  width: 9px;
+  height: 18px;
+  margin-bottom: -6px;
   position: relative;
   background: url('chrome://browser/skin/places/toolbarDropMarker.png') 50% 50% no-repeat;
 }
 
 .toolbar-drop-indicator-bar {
   visibility: hidden;
-  height: 16px;
-  margin-bottom: -16px;
-  margin-left: 4px;
+  height: 18px;
+  margin-bottom: -18px;
   position: relative;
 }
 

browser/themes/winstripe/browser/browser.css

@@ -1604,18 +1604,17 @@ toolbar[mode="text"] > #window-controls > toolbarbutton > .toolbarbutton-text {
 
 /* Personal toolbar */
 .toolbar-drop-indicator {
-  height: 16px;
-  width: 5px;
-  margin-bottom: -8px;
+  width: 9px;
+  height: 18px;
+  margin-bottom: -6px;
   position: relative;
   background: url("chrome://browser/skin/places/toolbarDropMarker.png") 50% 50% no-repeat;
 }
 
 .toolbar-drop-indicator-bar {
   visibility: hidden;
-  height: 16px;
-  margin-bottom: -16px;
-  margin-left: 4px;
+  height: 18px;
+  margin-bottom: -18px;
   position: relative;
 }