From cec78ed34807daee98e6bee7a47067f2a036f978 Mon Sep 17 00:00:00 2001
From: Dan Stillman <dstillman@zotero.org>
Date: Thu, 1 Feb 2007 02:19:31 +0000
Subject: [PATCH] Allow double-click on a saved search to edit

---
 chrome/content/zotero/overlay.js  | 36 +++++++++++++++++++------------
 chrome/content/zotero/overlay.xul |  1 +
 2 files changed, 23 insertions(+), 14 deletions(-)

diff --git a/chrome/content/zotero/overlay.js b/chrome/content/zotero/overlay.js
index adadd5a62..e5347ee5f 100644
--- a/chrome/content/zotero/overlay.js
+++ b/chrome/content/zotero/overlay.js
@@ -381,7 +381,7 @@ var ZoteroPane = new function()
 			height = height + 121;
 		}
 		
-		Zotero.debug('Setting Zotero pane minheight to ' + height);
+		//Zotero.debug('Setting Zotero pane minheight to ' + height);
 		zoteroPane.setAttribute('minheight', height);
 		
 		// Fix bug whereby resizing the Z pane downward after resizing
@@ -1125,22 +1125,30 @@ var ZoteroPane = new function()
 	// Adapted from: http://www.xulplanet.com/references/elemref/ref_tree.html#cmnote-9
 	function onDoubleClick(event, tree)
 	{
-		if (event && tree && (event.type == "click" || event.type == "dblclick"))
-		{
+		if (event && tree && event.type == "dblclick") {
 			var row = {}, col = {}, obj = {};
 			tree.treeBoxObject.getCellAt(event.clientX, event.clientY, row, col, obj);
-			// obj.value == cell/text/image
-			// TODO: handle collection double-click
-			if (obj.value && itemsView && itemsView.selection.currentIndex > -1)
-			{
-				var item = getSelectedItems()[0];
-				if(item && item.isNote())
-				{
-					document.getElementById('zotero-view-note-button').doCommand();
+			
+			// obj.value == 'cell'/'text'/'image'
+			if (!obj.value) {
+				return false;
+			}
+			
+			if (tree.id == 'zotero-collections-tree') {
+				var s = this.getSelectedSavedSearch();
+				if (s) {
+					this.editSelectedCollection();
 				}
-				else if(item && item.isAttachment())
-				{
-					viewSelectedAttachment();
+			}
+			else if (tree.id == 'zotero-items-tree') {
+				if (itemsView && itemsView.selection.currentIndex > -1) {
+					var item = getSelectedItems()[0];
+					if (item && item.isNote()) {
+						document.getElementById('zotero-view-note-button').doCommand();
+					}
+					else if (item && item.isAttachment()) {
+						this.viewSelectedAttachment();
+					}
 				}
 			}
 		}
diff --git a/chrome/content/zotero/overlay.xul b/chrome/content/zotero/overlay.xul
index a02abecb9..6d54e69e0 100644
--- a/chrome/content/zotero/overlay.xul
+++ b/chrome/content/zotero/overlay.xul
@@ -120,6 +120,7 @@
 						</toolbarbutton>
 					</toolbar>
 					<tree id="zotero-collections-tree" hidecolumnpicker="true" context="zotero-collectionmenu"
+						ondblclick="ZoteroPane.onDoubleClick(event, this);"
 						onselect="ZoteroPane.onCollectionSelected();" seltype="single"
 						ondragdrop="nsDragAndDrop.drop(event,ZoteroPane.getCollectionsView())"
 						ondraggesture="if (event.target.localName == 'treechildren') nsDragAndDrop.startDrag(event,ZoteroPane.getCollectionsView());"