lauzadis commented 4 months ago

Round 2!

Ensure only one toggle-content button is inserted
- I was sometimes seeing two toggle-content buttons
Insert a skip-to-main-content link on the first sideMenuPart
- Clicking a sideMenuPart would throw the tab index back to the top, meaning you have to tab all the way through to your current sideMenuPart before you will reach the skip-to-main-content link. I'm not sure if this is a browser-specific issue. This change adds a skip-link to the first sideMenuPart and the data-active sideMenuPart to ensure this case is covered.
Update invocation of ensureNavButtonInteractable function
- I wasn't seeing this function's effects on the prod docs, hopefully updating the way its invoked will fix this