Mis-nesting of aria roles on sidebar navigation

[ryan-garrett]

The sidebar navigation uses menuItem but without the correct grouping parent role, instead using a navigation role.
The use of the navigation role on the ul also changes the role of that element from an implicit list to an explicit non-list, so rendering the implicit child list items without a correctly named parent.

Before:

After:

Check yourself

• Code is linted
• Tested
• All new/updated code is covered with tests