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 html validator errors #3135

Merged
merged 2 commits into from
Apr 23, 2018
Merged

Fix html validator errors #3135

merged 2 commits into from
Apr 23, 2018

Conversation

mxxcon
Copy link
Contributor

@mxxcon mxxcon commented Apr 16, 2018

This PR addresses 2 issues reported by validator.w3.org:

According to https://w3c.github.io/html/single-page.html#statedef-http-equiv-content-type

A document must not contain both a <meta> element with an http-equiv attribute in the encoding declaration state and a <meta> element with the charset attribute present.

As such, I deleted the longer one.

Additionally, it complains:

Error: Bad value for attribute content on element meta: Expecting directive-name but found "↩".

As such, I removed line breaks.

mxxcon added 2 commits April 15, 2018 22:03
According to https://w3c.github.io/html/single-page.html#statedef-http-equiv-content-type
> A document must not contain both a meta element with an http-equiv attribute in the encoding declaration state and a meta element with the charset attribute present.
Validator throws error message:
Error: Bad value for attribute `content` on element `meta`: Expecting directive-name but found "↩".
@mxxcon mxxcon added enhancement Issue/PR contains enhancements to the overall code of the site. bug fix labels Apr 17, 2018
@mxxcon mxxcon requested review from fpigerre and a team April 19, 2018 03:17
@mxxcon mxxcon requested a review from a team April 23, 2018 17:49
@jamcat22 jamcat22 merged commit 1f5c27e into 2factorauth:master Apr 23, 2018
@mxxcon mxxcon deleted the html_errors branch April 23, 2018 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR contains enhancements to the overall code of the site.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants