Addresses #414, it is possible to edit the "extra" field and another field simultaneously
But don't break the independent note tags view Also fix another tags error in that view A lot of item pane logic really needs to be put in bindings, somehow...
This commit is contained in:
parent
d179ad4686
commit
268aad212b
|
@ -865,7 +865,9 @@ var ZoteroItemPane = new function()
|
||||||
function showEditor(elem)
|
function showEditor(elem)
|
||||||
{
|
{
|
||||||
// Blur any active fields
|
// Blur any active fields
|
||||||
_dynamicFields.focus();
|
if (_dynamicFields) {
|
||||||
|
_dynamicFields.focus();
|
||||||
|
}
|
||||||
|
|
||||||
//Zotero.debug('Showing editor');
|
//Zotero.debug('Showing editor');
|
||||||
|
|
||||||
|
@ -1038,7 +1040,10 @@ var ZoteroItemPane = new function()
|
||||||
|
|
||||||
// Return focus to items pane
|
// Return focus to items pane
|
||||||
if (!lastTag) {
|
if (!lastTag) {
|
||||||
document.getElementById('zotero-items-tree').focus();
|
var tree = document.getElementById('zotero-items-tree');
|
||||||
|
if (tree) {
|
||||||
|
tree.focus();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
|
@ -1049,7 +1054,10 @@ var ZoteroItemPane = new function()
|
||||||
target.blur();
|
target.blur();
|
||||||
|
|
||||||
// Return focus to items pane
|
// Return focus to items pane
|
||||||
document.getElementById('zotero-items-tree').focus();
|
var tree = document.getElementById('zotero-items-tree');
|
||||||
|
if (tree) {
|
||||||
|
tree.focus();
|
||||||
|
}
|
||||||
|
|
||||||
return false;
|
return false;
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue
Block a user