PageLayout.Content should not use main landmark by default.

[radglob]

PageLayout.Content will no longer render as main by default. Instead, end users can pass a specific as prop to set the rendered output.

Closeshttps://github.com/github/primer/issues/1586

Screenshots

Please provide before/after screenshots for any visual changes

Merge checklist

• Added/updated tests
• Added/updated documentation
• Tested in Chrome
• Tested in Firefox
• Tested in Safari
• Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

[changeset-bot]

🦋 Changeset detected

Latest commit: 4eb259c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@primer/react	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

size-limit report 📦

Path	Size
dist/browser.esm.js	121.56 KB (-0.01% 🔽)
dist/browser.umd.js	121.8 KB (-0.01% 🔽)

[TylerJDev]

Very light feedback. Let me know your thoughts!

Pending your comment on role usage, we'll need to adjust the documentation in PageLayout.mdx for PageLayout.Content.

[TylerJDev]

Left a comment for role usage, but I think this is good to go 🔥! There's some wording we might have to change in the docs around main usage now - but this can be a follow up PR later as it isn't a blocker.

[colebemis]