Take tag selector out of the tab order, for now
This restores the 4.0 behavior. It's not ideal, but neither is having to tab through thousands of items to get to the items pane (unless you toggle the tag selector closed first). Hopefully we can come up with a better solution to make the tags accessible.
This commit is contained in:
parent
c39bdaa25b
commit
6de7fdedf9
1 changed files with 7 additions and 3 deletions
|
@ -914,6 +914,9 @@
|
||||||
if (tagObj.type) {
|
if (tagObj.type) {
|
||||||
elem.setAttribute('tagType', tagObj.type);
|
elem.setAttribute('tagType', tagObj.type);
|
||||||
}
|
}
|
||||||
|
// Take the tags out of the tab order so that tabbing moves from the collections
|
||||||
|
// pane to the items pane
|
||||||
|
elem.setAttribute('tabindex', "-1");
|
||||||
return elem;
|
return elem;
|
||||||
]]></body>
|
]]></body>
|
||||||
</method>
|
</method>
|
||||||
|
@ -1157,14 +1160,15 @@
|
||||||
<label value="&zotero.tagSelector.noTagsToDisplay;"/>
|
<label value="&zotero.tagSelector.noTagsToDisplay;"/>
|
||||||
</box>
|
</box>
|
||||||
|
|
||||||
<html:div id="tags-box"/>
|
<html:div id="tags-box" tabindex="-1"/>
|
||||||
</deck>
|
</deck>
|
||||||
|
|
||||||
<vbox id="tag-controls">
|
<vbox id="tag-controls">
|
||||||
<hbox>
|
<hbox>
|
||||||
<textbox id="tags-search" flex="1" type="search" timeout="250" dir="reverse"
|
<textbox id="tags-search" flex="1" type="search" timeout="250" dir="reverse"
|
||||||
oncommand="document.getBindingParent(this).handleKeyPress(); event.stopPropagation()"
|
onkeypress="document.getBindingParent(this).onKeyPress(event)"
|
||||||
onkeypress="if (event.keyCode == event.DOM_VK_ESCAPE) { document.getBindingParent(this).handleKeyPress(true); }"/>
|
oncommand="document.getBindingParent(this).onCommand(event)"
|
||||||
|
tabindex="-1"/>
|
||||||
<toolbarbutton id="view-settings-menu" tooltiptext="&zotero.toolbar.actions.label;"
|
<toolbarbutton id="view-settings-menu" tooltiptext="&zotero.toolbar.actions.label;"
|
||||||
image="chrome://zotero/skin/tag-selector-menu.png" type="menu">
|
image="chrome://zotero/skin/tag-selector-menu.png" type="menu">
|
||||||
<menupopup id="view-settings-popup"
|
<menupopup id="view-settings-popup"
|
||||||
|
|
Loading…
Reference in a new issue