-
Notifications
You must be signed in to change notification settings - Fork 5
Check Web Accessibility #6
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
Comments
I'll have to look up the accessibility guidelines. Let's do this check once Dr. Lambrecht's dev server is up and running. |
I agree this isn't a primary concern right now. I also already increased the font from 14-16px. |
We need to revisit this. Please take a look particularly at the first two links, which indicate some issues, most should be fixable. |
99% of the accessibility "errors" presented by these analyzers is just how Bootstrap uses tags in a broad manner. |
I can make some changes to cut down on them. But you are right I don't On Sun, Jul 12, 2015 at 3:27 AM Ritwik Gupta notifications@github.com
|
All of the empty links that are being flagged for not having text do have title text. From what I have been looking up this is acceptable for accessibility. |
Hi @JoshuaRogan the title attribute (as far as I've tested) isn't read aloud on screenreaders and is meant to be an accompaniment to actual text inside the tags, not instead of. The W3C mention "For each anchor element that has a title attribute, check that the title attribute together with the link text describes the purpose of the link." Scott O'Hara wrote an amazingly thorough article on the complexities of the |
We need to check for web accessibility:
http://wave.webaim.org
http://achecker.ca/checker/
http://www.cynthiasays.com
http://fae20.cita.illinois.edu
Anecdotally, a visually impaired staff member said the font size was "small" but was able to increase it easily.
The text was updated successfully, but these errors were encountered: