fix: adds accessibility to code-group component

[lbenie]

Summary

Fixes the code-group component by wrapping the buttons in a list for accessibility purpose.

What kind of change does this PR introduce? (check at least one)

• Bugfix
• Feature
• Code style update
• Refactor
• Docs
• Build-related changes
• Other, please describe:

If changing the UI of default theme, please provide the before/after screenshot:

Before on the left
After on the right

Does this PR introduce a breaking change? (check one)

• Yes
• No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

• When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

• Chrome
• Firefox
• Safari
• Edge
• IE

If adding a new feature, the PR's description includes:

• A convincing reason for adding this feature
• Related documents have been updated
• Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

[d-pollard]

Do you happen to have an a11y resource that would explain how this improves accessibility vs what was previously there?

Other than the comment I made, it looks good!

[lbenie]

Do you happen to have an a11y resource that would explain how this improves accessibility vs what was previously there?

Other than the comment I made, it looks good!

Sure!

Some guidelines about list usage in regards to WCAG AA

https://www.a11yproject.com/checklist/#use-list-elements-lesscodegreaterolless/codegreater-lesscodegreaterulless/codegreater-and-lesscodegreaterdlless/codegreater-elements-for-list-content

Since the component is listing a bunch of related buttons, it makes more sense to group them under a list tag.

Although thinking on the nav element should be removed since it relates to navigation links. I'll update the PR to remove the nav and fix the CSS

[lbenie]

@d-pollard

I've updated the PR with the requested changes let me know if I need to do something else :)

[d-pollard]

Looks good! Thank you for making these changes!

[bencodezen]

Thanks @lbenie for your work on this! Appreciate it!