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

Keyboard Focus and Accessibility Focus should be the same #9826

Closed
chiaramooney opened this issue Apr 12, 2022 · 1 comment · Fixed by #9840
Closed

Keyboard Focus and Accessibility Focus should be the same #9826

chiaramooney opened this issue Apr 12, 2022 · 1 comment · Fixed by #9840

Comments

@chiaramooney
Copy link
Contributor

Problem Description

According to UIA and Narrator team, on Windows, keyboard focus and accessibility focus should always point to the same element. Currently in RNW, when a focusable element is passed accessible=false, the element is removed from the UIA tree but remains keyboard focusable. Thus the user, when tabbing to an element with accessible=false with a screen reader enabled, can see keyboard focus move to the element and accessibility focus jump to the app window. Then, the screen reader announces <app name> window creating a poor/disorienting screen reader experience.

Steps To Reproduce

Run the Playground App.
Edit a sample page to include a TouchableHighlight control and with the accessible set to false.
Enable narrator.
Tab to TouchableHighlight.

Expected Results

When an element is passed accessible=false, it should no longer be focusable leading both keyboard focus and the screen reader to skip over the element.

CLI version

N/A

Environment

N/A

Target Platform Version

10.0.19041

Target Device(s)

Desktop

Visual Studio Version

Visual Studio 2019

Build Configuration

Debug

Snack, code example, screenshot, or link to a repository

No response

@ghost ghost added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Apr 12, 2022
@AgneLukoseviciute
Copy link
Contributor

related: #8875

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

Successfully merging a pull request may close this issue.

4 participants