-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
WCAG 2.0 failure on buttons #518
Comments
Sure, I would consider adding this. Could you provide me a reference to the guideline? I don't understand why a button needs a title or label if it already has text. |
No problem, here's the link to the spec: I have just tested again and noticed that the button text values are added only when you open a colorbox. Because the HTML for the wrapper is created on page load, the buttons don't get properly labelled until then. |
Oh ok. I'm not too worried about them being empty at the beginning because I plan on replacing that soon so that it's only created on demand. |
Excellent! I've got a patch in to resolve it on my end so I'm happy. Once on demand makes it into a release I'll be a happy boy :-) |
Duplicate of #765. |
The replacement of div controls with buttons causes Colorbox to fail WCAG 2.0 validation because they do not have a title or label. I've tested adding labels successfully, is this something you would be willing to add in? Because the controls are all image replaced this shouldn't cause any conflicts.
The text was updated successfully, but these errors were encountered: