-
Notifications
You must be signed in to change notification settings - Fork 110
Adds a page about accessibility to the designing bitcoin products section #821
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
Conversation
This is currently still a work in progress.
Add as assignee |
Just to show how to make commits.
Small edit to test.
1) Updated explanations of disability types and included links to statistics. 2) Added in a link on how to test your apps accessibility.
Tried to edit for clarity. Added and updated some links.
1) Updated correct link to quadriplegic using phone 2) Changed heading to "Categories of accessibility needs" 3) Added paragraph about Illiteracy 4$ Added statistic source link to Hearing paragraph
…itcoinDesign/Guide into feature/new-accessibility-page
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.
Great work! Left a lot of grammar and style edits. Also suggesting we add a modal option to the example image carousels.
Just realized this was in draft mode. Sorry for providing such hyper-detailed feedback early when you may not have been ready for it. |
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Co-authored-by: Stephen DeLorme <stephen@stephendelorme.com>
Really appreciate your thorough feedback @sbddesign, very much appreciated. I've applied all those suggestions you've made. |
@sbddesign I think all your feedback is addressed and the page is ready for another review. |
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.
🔥
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 LGTM, great work on this page guys. I learnt a lot reading it
Adds general information about the importance of accessibility, practical tips to get started, and some useful resources.
🙉Check the preview🥸
Closes #754.