-
Notifications
You must be signed in to change notification settings - Fork 171
Adding in some accessibility-related language #898
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
Adding in some accessibility-related language #898
Conversation
Lookin good, but could also use a quick tidy 🙏 |
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
@marcoscaceres I merged in the removed color processing steps too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All changes were resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great... let's see if we can convince the A11y folks to export things properly :)
Blocked on getting the A11y folk to export their definitions... |
@marcoscaceres I went old school on the definitions here to unblock merging this PR. Let me know if I missed anything. Respec isn’t complaining so it may be a good stopgap. |
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Co-authored-by: Marcos Cáceres <marcos@marcosc.com>
Closes #860
This change (choose one):
changes normative sections without changing behavior)
Commit message:
Taking a crack at adding accessibility info. The only outstanding item is adding a recommendation for screenshot
label
s once ImageResource is integrated.Preview | Diff