fix: 943-audit-avt1-homepage-content-not-in-landmark

[andreancardona]

Closes #943

please note that the branch name does not match the PR / fix - personal mistake

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/carbon-design-system/gatsby-theme-carbon/2tsbfi1p2
✅ Preview: https://gatsby-theme-ca-git-fork-andreancardona-943-audit-avt1-h-8dcc31.carbon-design-system.vercel.app

[vpicone]

@andreancardona did this resolve DAP issue? Hopefully without out creating new ones haha.

[andreancardona]

@andreancardona did this resolve DAP issue? Hopefully without out creating new ones haha.

@vpicone @jnm2377 Looks like it did! :)

[jnm2377]

If you click on the main section when the sidenav is open, there's like a focus outline on part of it. Is it meant to do that?

[andreancardona]

If you click on the main section when the sidenav is open, there's like a focus outline on part of it. Is it meant to do that?

@jnm2377 from what I can tell that's what it currently does / that is the current behavior - I'm not sure though 🤷‍♀️

[jnm2377]

ahhh gotcha @andreancardona hmm.. I'm not sure we want a focus outline... but since it's an existing issue, I'm fine with merging :)