www

commit c906c8e0bec0d17e036153cb68515968bfc2144e
parent edf113f18424f2ba2098d5b871175c5a544ff045
Author: Simon Kornblith <simon@simonster.com>
Date:   Sat, 26 Mar 2011 21:55:23 +0000

fix doubled separators


Diffstat:
M
chrome/content/zotero/locateMenu.js
 | 
12
+++++-------

1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/chrome/content/zotero/locateMenu.js b/chrome/content/zotero/locateMenu.js
@@ -52,17 +52,17 @@ var Zotero_LocateMenu = new function() {
 			if(availableEngines.length) {
 				_addLocateEngines(locateMenu, availableEngines, true);
 			}
-			
-			// add separator at end if necessary
-			if(locateMenu.lastChild.tagName !== "menuseparator") {
-				locateMenu.appendChild(document.createElement("menuseparator"));
-			}
 		} else {
 			// add "no items selected"
 			menuitem = _createMenuItem(Zotero.getString("pane.item.selected.zero"), "no-items-selected");
 			locateMenu.appendChild(menuitem);
 			menuitem.disabled = true;
 		}
+					
+		// add separator at end if necessary
+		if(locateMenu.lastChild.tagName !== "menuseparator") {
+			locateMenu.appendChild(document.createElement("menuseparator"));
+		}
 		
 		// add installable locate menus, if there are any
 		if(window.Zotero_Browser) {
@@ -72,8 +72,6 @@ var Zotero_LocateMenu = new function() {
 		}
 		
 		if(installableLocateEngines.length) {
-			locateMenu.appendChild(document.createElement("menuseparator"));
-			
 			for each(var locateEngine in installableLocateEngines) {
 				var menuitem = document.createElement("menuitem");
 				menuitem.setAttribute("label", locateEngine.label);