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 w3c validator errors #161

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mvitz
Copy link
Contributor

@mvitz mvitz commented Mar 13, 2015

This PR fixes all w3c validation errors/warnings as described in #130.

@mvitz mvitz force-pushed the 130-fix-w3c-validator-errors branch 2 times, most recently from 364c9b7 to 340c6bc Compare April 8, 2015 20:28
@mvitz mvitz force-pushed the 130-fix-w3c-validator-errors branch from 340c6bc to 00afd62 Compare April 22, 2015 15:04
@mvitz mvitz force-pushed the 130-fix-w3c-validator-errors branch from 00afd62 to 27264cf Compare May 1, 2015 11:19
@mvitz mvitz force-pushed the 130-fix-w3c-validator-errors branch from 27264cf to bd8bf42 Compare November 23, 2015 20:08
Specifying the charset prevents the browser from guessing the charset to
use.

Relates to innoq#130.
Instead of a header with role navigation now the HTML5 element nav is
used.

Relates to innoq#130.
Removes warnings from w3c validator that label and input are not valid
inside an a element. Because preferences aren't links anyway this
increases semantic as well.

Relates to innoq#130.
By default hiccup generates an id for every form field. In our case
(more than one form with method DELETE on the same page) this leads to
non unique ids which leads to w3c validation errors.

Relates to innoq#130.
Workaround for weavejester/hiccup#109.
Only inline elements may be contained whithin inline elements. Because
form is not an inline element this leads to w3c validation errors.

Relates to innoq#130.
@mvitz mvitz force-pushed the 130-fix-w3c-validator-errors branch from bd8bf42 to d477a31 Compare March 10, 2016 09:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant