scriptoriumadmin/zotero
1
0
Fork 0
mirror of https://github.com/zotero/zotero.git synced 2026-10-08 03:08:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix itemTreeView.rememberSelection() to find children of unopened parents as well as top-level items

Browse source 
This makes the quicksearch considerably more useful, as you can search for a child item in the library, select it, and then clear the search to view the item in context. (Works for multiple items as well, even of different parents.)
This commit is contained in:
Dan Stillman 2007-01-02 13:15:31 +00:00
parent 0abda111a6
commit 8710875fdb
1 changed files with 21 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
chrome/content/zotero/xpcom/itemTreeView.js
View file

@ -612,7 +612,7 @@ Zotero.ItemTreeView.prototype.selectItem = function(id)
// No parent -- it's not here
return false;
}
this.toggleOpenState(this._itemRowMap[item.getSource()]); //opens the parent of the item
this.toggleOpenState(this._itemRowMap[parent]); //opens the parent of the item
row = this._itemRowMap[id];
}
@ -755,6 +755,7 @@ Zotero.ItemTreeView.prototype.saveSelection = function()
savedSelection.push(this._getItemAtRow(j).ref.getID());
}
}
return savedSelection;
}
@ -766,8 +767,26 @@ Zotero.ItemTreeView.prototype.rememberSelection = function(selection)
this.selection.clearSelection();
for(var i=0; i < selection.length; i++)
{
if(this._itemRowMap[selection[i]] != null)
if (this._itemRowMap[selection[i]] != null) {
this.selection.toggleSelect(this._itemRowMap[selection[i]]);
}
// Try the parent
else {
var item = Zotero.Items.get(selection[i]);
if (!item) {
continue;
}
var parent = item.getSource();
if (!parent) {
continue;
}
if (this._itemRowMap[parent] != null) {
this.toggleOpenState(this._itemRowMap[parent]);
this.selection.toggleSelect(this._itemRowMap[selection[i]]);
}
}
}
}