Remove role="main" to ensure 'NPM Docs' page contains only one main landmark

npm / documentation

Documentation for the npm registry, website, and command-line interface.

https://docs.npmjs.com/

Creative Commons Attribution 4.0 International

414 stars 2.86k forks source link

Remove role="main" to ensure 'NPM Docs' page contains only one main landmark #1354

Closed maitxn closed 2 weeks ago

maitxn commented 3 weeks ago

This pull request removes the role="main" attribute from the PageElement component in the src/page.js file to ensure there is only one main landmark on the 'NPM Docs' page.

Before

NPM Docs page with more than one main landmark

After

NPM Docs page with only one main landmark

References

https://github.com/github/accessibility-audits/issues/9212

maitxn commented 3 weeks ago

Hi @wraithgar can you please merge this PR to prod?