Update landmarks and labels

[becka11y]

The table of contents area is not labeled as a landmark. Adding this would allow screen reader users to move to the ToC using the landmark features of screen readers (list landmarks or navigate from landmark to landmark). I would suggest labeling the ToC with a role=navigation and an aria-label="Table of Contents" or add an id to the H2 that represents the Table of Contents text and use aria-labelledby. There is already a resource string for the "Table of Contents" text.

Since the keyboard shortcuts refer to the Toolbar it should be labeled as such. Right now the toolbar does have a role=navigation, I would add a label of "Toolbar". This would also need an added string resource for translation. Better yet would be to add the keyboard arrow key navigation and make it true toolbar with a role of toolbar. But, that is a stretch goal.

[danielweck]

Fixed by:
70fd6fb

[danielweck]

New issue recorded for arrow keys navigation of toolbar:

#100