-
Notifications
You must be signed in to change notification settings - Fork 1
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
Validate output against WCAG 2.1 #5
Comments
What kind of output are you trying to validate exactly? The text shouldn't have a colour by default, and should just inherent the colour provided by the theme. From what I understood from Viv, the text shouldn't be on top of the progress bar, those two should be separate. The progress bar should inherit the text colour as well, unless otherwise. |
The progress bar works on a color base. I need to ensure that the colors use a sufficient contrast. In addition, visually impaired users that use a scree reader must be able to understand the progress bar as well. Therefore, I need to add additional text for the screen reader. |
I'm not sure I'm following. To know if you have enough contrast, you need to know the colour offhand, which is not possible here given you shouldn't be providing a colour for the progress bar. |
I'd (if I'm understanding the requirements correctly), this issue should be blocked by #6.
|
Your explanation makes sense, @senadir. I'll deprioritize this issue accordingly. |
To make the plugin inclusive, the output needs to fulfil the WCAG 2.1 AA standard.
Tasks
<div class="progress-container">...</div>
with<progress>...</progress>
aria-valuemin
,aria-valuemax
andaria-valuenow
to<progress>
elementSources
The text was updated successfully, but these errors were encountered: