-
Notifications
You must be signed in to change notification settings - Fork 289
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
Fix accessibility labels on profile page #1120
Comments
https://damus.io/note1lv8nzkfe6ns3smga59c8rdgsu56az7veh435xc9vzrs390xhg7gqmpdc6x |
I did a quick test for Global Accessibility Awareness Day and found a lot of issues, even just on the profile screen. It's pretty much all labels, along with some grouping. It might seem like a lot, but this stuff is pretty straightforward to fix. |
Thank you for the detailed feedback. Shared with dev team. |
@BenGWeeks this is the accessibility labels work @GBKS mentioned during today's Damus devs call. Want to take this one 🙏 ? |
Yeah, I'll take this one. |
If you'd like more info on testing this stuff, we have resources on the Bitcoin Design Guide accessibility page. Here's a short video on I usually test things. |
Did you use the "Accessibility Inspector" in XCode? Using Audit does not finish the audit for me ... |
No, I just grab my phone, enable VoiceControl and VoiceOver from the accessibility short-cut menu (like in the linked video) and tap around the app. Basically testing what the end-users have in their hands. |
Related #1297 |
Work in progress
https://damus.io/note15n62vjahkxz4u97a59yskqn9g4q8xrdlnz3ahapz3wrsnfa7nx5sc8avmg
The text was updated successfully, but these errors were encountered: