fix(ui): fixed sidenav active state

[robinrendle]

This secondary sidenav bar is pretty complex. We want it to be sticky when scrolling the main content area, but also let you scroll within the menu itself when the browser window cuts off the items. In my previous change I didn’t take into consideration that the scroll overflow would cut off the active state, so I’m fixing that here by moving the padding from the outer nav to the sticky positioned component.

Before

After

[evanpurkhiser]

This is a good fix