-
Notifications
You must be signed in to change notification settings - Fork 125
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
Code Examples: Validate HTML and CSS #174
Comments
I think we better start with the simple addition of an .editorconfig file and go from there. We need to see how that impacts the people working on this. To be honest, I'm not sure about adding dependencies on this. (I feel) this is stuff is not all that complicated. On the other hand, automated HTML validation and the like would be a plus. |
I have validated all examples (the code still needs cleaning up, but that is a different issue): fadbd7f @karlgroves I'll let you know if we're going forward with adding something like grunt/gulp/npm scripts. |
Definitely agree with starting small, but IMO you could/ should auto beautify to enforce your coding standards and require that all PRs pass the validity checks. |
I agree, have filed an issue: #202 |
add doc-dedication to aside dpub allowances
As Karl pointed out, all code examples should be validated.
For those wondering: I've removed all
role="main"
attributes. These trigger a warning in the validator. Besides, the main element should suffice; there are no other structural landmarks.The text was updated successfully, but these errors were encountered: