-
Notifications
You must be signed in to change notification settings - Fork 62
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
TodoMVC workloads - A11y audit #112
Labels
Comments
Merged
I think this is a blocker to release v3. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
While updating the TodoMVC applications, we noticed a few a11y related issues.
For example inconsistency in using inputs with and without labels, or using inputs without a form and therefore having to manage keyboard events more extensively.
These inconsistencies don't interfere with the test results, nor do they speed up the workloads once addressed most likely.
That said, I believe that since our workloads are supposed to represent apps on the web, we should also advocate for best practices where it makes sense. This will hopefully also encourage other engineers to follow similar patterns when developing their own applications.
The text was updated successfully, but these errors were encountered: