[a11y]: HeaderSideNavItems should use a ul rather than div

[samrobbins85]

Package

carbon-components-react

Browser

Chrome

Operating System

MacOS

Package version

7.50.0

React version

16.14.0

Automated testing tool and ruleset

axe devtools

Assistive technology

No response

Description

Using the method described in storybook of <HeaderSideNavItems> with <HeaderMenuItem>s nested inside you get a div (HeaderSideNavItems) containing lis (HeaderMenuItem). Instead HeaderSideNavItems should be a list

WCAG 2.1 Violation

Success Criterion 1.3.1 Info and Relationships

CodeSandbox example

https://codesandbox.io/s/ibm-sidenav-a11y-2-yrwlo?file=/package.json

Steps to reproduce

Narrow the window such that the mobile menu comes up, open this menu and inspect the top section, it contains a div containing lis

Code of Conduct

• I agree to follow this project's Code of Conduct
• I checked the current issues for duplicate problems