www

commit f0badddaf18ade0f319c5c2561f07ac35c763da9
parent 511eda7b8680044646e40bb0b6e11c1f949a0c3e
Author: Dan Stillman <dstillman@zotero.org>
Date:   Thu, 31 Aug 2006 21:18:41 +0000

Added support for deleting with forward-delete key, which may have worked before the treeviews were moved into XPCOM but didn't anymore


Diffstat:
M
chrome/chromeFiles/content/scholar/overlay.xul
 | 
5
+++--
M
chrome/chromeFiles/content/scholar/xpcom/collectionTreeView.js
 | 
5
-----
M
chrome/chromeFiles/content/scholar/xpcom/itemTreeView.js
 | 
6
++----

3 files changed, 5 insertions(+), 11 deletions(-)
diff --git a/chrome/chromeFiles/content/scholar/overlay.xul b/chrome/chromeFiles/content/scholar/overlay.xul
@@ -75,7 +75,7 @@
 					onselect="ScholarPane.onCollectionSelected();" seltype="single"
 					ondragdrop="nsDragAndDrop.drop(event,ScholarPane.getCollectionsView())"
 					ondraggesture="if (event.target.localName == 'treechildren') nsDragAndDrop.startDrag(event,ScholarPane.getCollectionsView());"
-					onkeypress="if(event.keyCode == event.DOM_VK_BACK_SPACE){ ScholarPane.deleteSelectedCollection(); return false; }"
+					onkeypress="if (event.keyCode == event.DOM_VK_BACK_SPACE || event.keyCode == event.DOM_VK_DELETE){ ScholarPane.deleteSelectedCollection(); return false; }"
 					flex="1">
 					<treecols>
 						<treecol
@@ -111,7 +111,8 @@
 				<tree 
 					id="items-tree" context="scholar-itemmenu"
 					ondblclick="ScholarPane.onDoubleClick(event, this);"
-					enableColumnDrag="true" onkeypress="if(event.keyCode == event.DOM_VK_BACK_SPACE){ ScholarPane.deleteSelectedItem(); return false; }"
+					enableColumnDrag="true"
+					onkeypress="if (event.keyCode == event.DOM_VK_BACK_SPACE || event.keyCode == event.DOM_VK_DELETE){ ScholarPane.deleteSelectedItem(); return false; }"
 					onselect="ScholarPane.itemSelected();"
 					ondraggesture="if (event.target.localName == 'treechildren') nsDragAndDrop.startDrag(event,ScholarPane.getItemsView());"
 					ondragover="nsDragAndDrop.dragOver(event,ScholarPane.getItemsView())"
diff --git a/chrome/chromeFiles/content/scholar/xpcom/collectionTreeView.js b/chrome/chromeFiles/content/scholar/xpcom/collectionTreeView.js
@@ -432,23 +432,18 @@ Scholar.CollectionTreeCommandController = function(tree)
 
 Scholar.CollectionTreeCommandController.prototype.supportsCommand = function(cmd)
 {
-	return (cmd == 'cmd_delete');
 }
 
 Scholar.CollectionTreeCommandController.prototype.isCommandEnabled = function(cmd)
 {
-	return (cmd == 'cmd_delete' && this.tree.view.selection.count > 0);
 }
 
 Scholar.CollectionTreeCommandController.prototype.doCommand = function(cmd)
 {
-	if(cmd == 'cmd_delete')
-		ScholarPane.deleteSelectedCollection();
 }
 
 Scholar.CollectionTreeCommandController.prototype.onEvent = function(evt)
 {
-	
 }
 
 ////////////////////////////////////////////////////////////////////////////////
diff --git a/chrome/chromeFiles/content/scholar/xpcom/itemTreeView.js b/chrome/chromeFiles/content/scholar/xpcom/itemTreeView.js
@@ -634,20 +634,18 @@ Scholar.ItemTreeCommandController = function(tree)
 
 Scholar.ItemTreeCommandController.prototype.supportsCommand = function(cmd)
 {
-	return (cmd == 'cmd_selectAll' || cmd == 'cmd_delete');
+	return (cmd == 'cmd_selectAll');
 }
 
 Scholar.ItemTreeCommandController.prototype.isCommandEnabled = function(cmd)
 {
-	return (cmd == 'cmd_selectAll' || (cmd == 'cmd_delete' && this.tree.view.selection.count > 0));
+	return (cmd == 'cmd_selectAll');
 }
 
 Scholar.ItemTreeCommandController.prototype.doCommand = function(cmd)
 {
 	if(cmd == 'cmd_selectAll')
 		this.tree.view.selection.selectAll();
-	else if(cmd == 'cmd_delete')
-		ScholarPane.deleteSelectedItem();
 }
 
 Scholar.ItemTreeCommandController.prototype.onEvent = function(evt)