-
Notifications
You must be signed in to change notification settings - Fork 116
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
Give buttons more inspiring names #2206
Conversation
|
|
@keithamus: As checked on all these 5 issues, it is still assigned to one of the testers. We have a process of two level review on bugs logged and unassign it when review is complete. We will review these bugs and update the details accordingly. |
f9930ea
to
d36a9d6
Compare
What are you trying to accomplish?
We've had a slew of new accessibility violations raised on our various buttons for not being descriptive enough. I personally think this is a mistake because the issues claim that
button Button
is not descriptive, ignoring the fact that the Button is an example button with the literal nameButton
- it is as descriptive as the visual representation... alas I digress...Giving these buttons more inspiring names will demonstrate clearly that ATs describe the visuals, and there will be no more ambiguity around
button Button
.Screenshots
Integration
List the issues that this change affects.
Closes https://github.com/github/accessibility-audits/issues/5423
Closes https://github.com/github/accessibility-audits/issues/5419
Closes https://github.com/github/accessibility-audits/issues/5418
Closes https://github.com/github/accessibility-audits/issues/5414
Closes https://github.com/github/accessibility-audits/issues/5412
Risk Assessment
What approach did you choose and why?
Anything you want to highlight for special attention from reviewers?
Accessibility
Merge checklist
Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.