-
Benachrichtigungen
You must be signed in to change notification settings - Fork 160
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
Adding in some accessibility-related language #898
Conversation
Lookin good, but could also use a quick tidy 🙏 |
Co-authored-by: Marcos Cáceres <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
@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 <[email protected]>
Co-authored-by: Marcos Cáceres <[email protected]>
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