www

commit 1e90528937f1ed73563f127a0776d3d060922d10
parent 914f29b8ac442726534fe62811293e6d49f0a20e
Author: Simon Kornblith <simon@simonster.com>
Date:   Thu, 25 Aug 2011 21:24:51 +0000

Don't build item type menu twice


Diffstat:
M
chrome/content/zotero/zoteroPane.js
 | 
14
+++++++++-----

1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/chrome/content/zotero/zoteroPane.js b/chrome/content/zotero/zoteroPane.js
@@ -87,10 +87,10 @@ var ZoteroPane = new function()
 	
 	const COLLECTIONS_HEIGHT = 32; // minimum height of the collections pane and toolbar
 	
-	var self = this;
-	var _loaded = false;
-	var titlebarcolorState, titleState, observerService;
-	var _reloadFunctions = [];
+	var self = this,
+		_loaded = false, _madeVisible = false,
+		titlebarcolorState, titleState, observerService,
+		_reloadFunctions = [];
 	
 	/**
 	 * Called when the window containing Zotero pane is open
@@ -366,7 +366,11 @@ var ZoteroPane = new function()
 			return false;
 		}
 		
-		this.buildItemTypeSubMenu();
+		if(!_madeVisible) {
+			this.buildItemTypeSubMenu();
+		}
+		_madeVisible = true;
+		
 		this.unserializePersist();
 		this.updateToolbarPosition();
 		this.updateTagSelectorSize();