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

Accessibility Fixes Round 2 #464

Closed
8 tasks done
andrewmcgivery opened this issue Dec 9, 2014 · 2 comments · Fixed by #468
Closed
8 tasks done

Accessibility Fixes Round 2 #464

andrewmcgivery opened this issue Dec 9, 2014 · 2 comments · Fixed by #468
Milestone

Comments

@andrewmcgivery
Copy link
Contributor

Homepage

  • User Avatar missing alt text in header
  • Report Bug form radio buttons should be contained within a fieldset... will need CSS styling to more or less remove the default browser styles to keep the current look
  • If user has reported no issues, the "Nothing reported yet!" text is size 0 font

About

  • Missing H1... Should we make a h1 with .wc-sronly class applied or change "About" heading to H1 and then adjust rest of headings accordingly?

Privacy

  • Missing H1... Should we make a h1 with .wc-sronly class applied or change "Webcompat.com Privacy Policy" heading to H1 and then adjust rest of headings accordingly?
  • Jumps from H2 to H4... should change H4's to H3's and fix styles accordingly.

Contributors

  • Images should have more specific alt text other than "Browser Snapshot"
  • Jumps from H1 to H3's.... should change the H3's to H2's and style accordingly

Overall, we're in good shape. Just a few minor fixes and we'll be error free and warning free! (With the exception of a couple false positives)

@miketaylr
Copy link
Member

Sweet, thanks for tracking this. :)

@andrewmcgivery
Copy link
Contributor Author

Going to start working on this.

Luckily the form changes on the homepage don't appear to need any CSS fixed! Should just work.

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 a pull request may close this issue.

2 participants