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.

Sign up for GitHub

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

Jump to bottom

[HIGH] HTML validation #284

Open
shawna-slh opened this issue Jul 25, 2022 · 1 comment
Open

[HIGH] HTML validation #284

shawna-slh opened this issue Jul 25, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@shawna-slh
Copy link
Contributor

See https://validator.w3.org/nu/?doc=https%3A%2F%2Fdeploy-preview-248--wai-course-list.netlify.app%2Fcourses%2Flist%2F&showoutline=yes
@leticiaseixas leticiaseixas added the bug Something isn't working label Jul 26, 2022
leticiaseixas added a commit that referenced this issue Jul 26, 2022
@leticiaseixas
#284
7928248
@leticiaseixas
Copy link
Collaborator

Hey @shawna-slh ,

Thank you for poiting that out.
I fixed most of the errors. However, there are still three issues. Two of them coming from the main template - and also replicated in other pages through the WAI Website:

  1. Invalid character in the footer email link (Bad value mailto:wai-eo-editors@w3.org?subject=[en]%20Course%20List for attribute href on element a: Illegal character in query: [ is not allowed.);

  2. Using the expand/collapse element without a heading as the visible triggering text, generates a <p> element (Element p not allowed as child of element summary in this context.), as in the details for each course (other examples are also under Video: Conformance Evaluation Overview - https://www.w3.org/WAI/test-evaluate/conformance/wcag-em/#video or under Video: Involving Users in Web Accessibility Overview - https://www.w3.org/WAI/planning/).

And the last one, I don't know if there is a better way to do this on the current site build:

  1. To include another stylesheet in this resource, we call it with a <style> element at the beginning of the .md document. However, after the build, this reference is called out of the <head> section (Element style not allowed as child of element main in this context. ). Is there another way to include the stylesheet reference?

mugiwarafx added a commit to mugiwarafx/wai-course-list that referenced this issue Sep 19, 2022
@mugiwarafx
fix w3c#284 styles
c55ea18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@leticiaseixas @shawna-slh