Bookmark all can be activated by spacebar

[rladdusaw]

Resolves #4760

[coveralls]

coverage: 95.697%. remained the same
when pulling a249082 on 4760-bookmark-all-tab-order
into a7eb66c on main.

[sandbergja]

Thanks, @rladdusaw ! I had one question here.

[sandbergja]

Could we use a button, rather than a div here? Then we could listen for a click event to catch either the space bar or the enter key. It would also get added to the tab order automatically, without having to specify tabindex.

[rladdusaw]

I gave this a quick test, but it messed up the styling pretty badly. I could do it, but it might take a while to sort out the css.

I added the enter key to the event listener.

[sandbergja]

Thanks for looking into this! I think the button would be better long term (we wouldn't need to remember the tabindex if we refactor this code in the future, for example), if you can get the CSS sorted. Another benefit is that the button would be navigable by assistive technologies that allow you to navigate by form control (like NVDA).