Trivial edit. Remove tabindex -1.
Created attachment 146946 [details] [review] Bug 33012: Remove tabindex from logout button Note: This is a trivial edit. But there is another logout link too in masthead with js-hide class and tabindex -1 which is rather mysterious to me ;) Test plan: Login on OPAC. Try to logout by tabbing to Logout button. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Trivial one liner. Self SO
It looks like you can also not tab to 'search history' in the same menu as the log out link. Looking at masthead.inc we have a 'ton' of tabindex=-1. Another example would be the list menu, that cannot be navigated by tabbing, but you can use 'arrow down'. The same does not work for the 'Welcome...' menu. This would unlock one option, but it seems to be a bigger issue even in the same menu.
I think in terms of accessibility we should investigate this further. I just felt not sure if your solution was the right one as it only fixed one of the items in this menu. I'll reopen with a different description.