fix(ui-shell): resolve html structure bugs

[emyarod]

Related #827

This PR updates the sidenav popovers to render inside li elements as well as removes inline stules in SideNavFlyoutMenu

note: the referenced issue will require an upstream update for iconbuttons/tooltips

Changelog

New

• side nav flyout menu classes for display styles

Changed

• rendered sidenav popover and tooltip html

Removed

• inline styles

Testing / Reviewing

confirm that the sidenav popovers and tooltips are rendered properly and check that the styles for the flyout menus are unchanged

[netlify]

✅ Deploy Preview for carbon-labs-web-components ready!

Name	Link
🔨 Latest commit	6a8cded
🔍 Latest deploy log	https://app.netlify.com/projects/carbon-labs-web-components/deploys/68f7ce0afed3150008102104
😎 Deploy Preview	https://deploy-preview-847--carbon-labs-web-components.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[netlify]

✅ Deploy Preview for carbon-labs-react ready!

Name	Link
🔨 Latest commit	6a8cded
🔍 Latest deploy log	https://app.netlify.com/projects/carbon-labs-react/deploys/68f7ce0a1fa70e000859be37
😎 Deploy Preview	https://deploy-preview-847--carbon-labs-react.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[alisonjoseph]

@emyarod I'm seeing this html now, which is still invalid, wasn't sure exactly what part of this needed an upstream change?

<li class=popover...>
   <li class=side-nav__item....>

[emyarod]

@alisonjoseph I thought that IconButton would need the as prop to pass into the underlying Tooltip for some of the sidenav items. I'll also look into the doubled li > li, but it should be fine if we're able to change the innermost li to a span or something right?

[alisonjoseph]

@emyarod yes, but it would only need to be a span I think when its part of the pinned rail, otherwise it needs to be an li because it won't be wrapped in the popover. Will need to check any aria roles on that also.

[alisonjoseph]

Getting closer, but html still isn't quite right. spans can't be direct children of a ul

and looks like the updates broke styling in the other views