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:
Dan Stillman 2017-07-20 02:59:18 -04:00
parent c39bdaa25b
commit 6de7fdedf9

View file

@ -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"