Skip to content
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

[Feature Request] : add a11y support in <ControlsVisibilityProvider /> for screen reader #2

Open
quentingrchr opened this issue Oct 17, 2023 · 0 comments · May be fixed by #3
Open
Assignees

Comments

@quentingrchr
Copy link

quentingrchr commented Oct 17, 2023

Issue:

Currently, the <ControlsVisibilityProvider /> has a visibility feature that is triggered either by the autoHideAfterDuration timer or by a tap (toggle) on the video. This doesn't work well with accessibility (a11y), as all the buttons cannot be read when the controls are not visible.

Solution:

  • We could add an "alwaysVisible" prop/parameter to override the "visible" prop of the <ControlsVisibilityProvider /> component.

  • We could incorporate a built-in screen reader detection feature into this library that would manage the visibility of the controls, ensuring that users with screen readers can use the player.

Additionally, some prop types, such as those in <ControlSlider/>, could be enhanced to accept a11y props like "accessibilityState," "accessibilityRole," or "accessibilityLabel."

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant