Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix HTML5 validation check (role attribute of body tag is unnecessary) #3495

Merged
merged 1 commit into from
Mar 3, 2017

Conversation

shibukawa
Copy link
Contributor

Subject:

Feature or Bugfix

  • Bugfix

Purpose

  • Fix HTML5 validator issue

Detail

Relates

@shibukawa shibukawa changed the base branch from stable to master March 3, 2017 05:48
@shibukawa shibukawa force-pushed the fix/html5_validation_error branch from ed45a8d to a34ebb6 Compare March 3, 2017 05:50
@tk0miya
Copy link
Member

tk0miya commented Mar 3, 2017

Note: HTML5 spec says:

Allowed ARIA role attribute values:
document role (default - do not set), application.
https://www.w3.org/TR/html5/sections.html#the-body-element

And it is not recommended.

@tk0miya tk0miya added this to the 1.6 milestone Mar 3, 2017
@tk0miya tk0miya merged commit 28b1ace into sphinx-doc:master Mar 3, 2017
@shibukawa shibukawa deleted the fix/html5_validation_error branch March 4, 2017 01:44
Blendify added a commit to readthedocs/sphinx_rtd_theme that referenced this pull request Jan 15, 2018
* Do not set the default role for the body tag

See https://www.w3.org/TR/html5/sections.html#the-body-element And:
sphinx-doc/sphinx#3495

* Remove nav role from nav tag
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants