-
Notifications
You must be signed in to change notification settings - Fork 19
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
☂️ Improve Accessibility Features #7
Comments
Hello @giolaq , I would like to pick the following tasks:
thanks |
Great! |
@giolaq for high-contrast mode theme would there be a UI element to switch between light and dark or it would be based on the device settings? What is expectation for Document accessibility features and usage? Is it document like :
|
IMHO the simplest way is to add a "Settings" screen accessible from the left side drawer where the user can access these features and enable/disable. Best option will be associate these to the device settings, but this it's much more complex as different TV OSs will have different way to enable it. For the documentation, you can add an ACCESSIBILITY.md and link it to the README, what do you think? |
I will add the "change theme" in settings. Any reason for creating ACCESSIBILITY.md? Can't we reuse README? We can add a section ACCESSIBILITY section and add all accessibility information under that. |
Ok, if it becomes too big we can move it to another file |
While working on this issue, I found that the d-pad is not working on Android emulator (the config I am using). I will investigate it more with this issue. I have did a test on Web and app is working fine. @giolaq you can assign me this too (as I am already testing all 2-3 platforms) "Test the app with screen readers on different platforms (Android TV, Fire TV, tvOS, Web)" |
Hi @Neha any updates on these tasks? Hacktoberfest is finishing on 31st, maybe we can release some task for other people to pickup |
@giolaq for theme switcher, what is your thought on adding the theme switcher in the left navigation? The reason is to have the theme switcher in just one click away. It is a good UX experience. |
Usually the TV apps have only options related to content, profile or navigation in the main screen. While the theme switcher is a good UX in the front page of a website, for TV apps such a switcher will add more noise to to UI. |
@giolaq , My suggestion is to break the tasks into separate issues. These could be separate issues:
Document accessibility features and usage in the README (this should be the subtask of all the above issues). What is the end goal of this task? Test the app with screen readers on different platforms (Android TV, Fire TV, tvOS, Web) lmk wyt |
That's why it's an Umbrella Issue with task associated and you have 3 assigned. "Test the app with screen readers on different platforms (Android TV, Fire TV, tvOS, Web)" This is about testing the screen readers on the majority of tv OSs out there, making sure we can support them or open other activities to support them |
Improve Accessibility Features
Description
We need to enhance the accessibility features of our React Native Multi-TV App to ensure it's usable by people with various disabilities. This will involve adding proper aria labels, ensuring correct tab order, and implementing high-contrast mode.
Tasks
Expected Outcome
The app should be fully navigable and usable with a screen reader and keyboard/remote control only. All important elements should have clear, descriptive labels.
Resources
The text was updated successfully, but these errors were encountered: