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

RNW Accessibility Issue: Flyout Focus Management Affecting Windows Narrator and Keyboard Navigation #14194

Open
sgmuiruri opened this issue Dec 10, 2024 · 3 comments
Assignees
Labels
Area: Flyout Area: Paper bug Needs: Attention 👋 An issue that had been tagged "Needs: Author Feedback" has received activity (label applied by bot) Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Parity: Windows RNW does not look or behave like non-RNW Windows
Milestone

Comments

@sgmuiruri
Copy link

Problem Description

flyout component fails to acheive focus when toggled unless clicked into. Adding focus refs for focus management and accessiblity props does not help resolve the issue either.

Steps To Reproduce

Issue can be easily reproduced by looking at the RN Gallery's flyout page.
launch windows narrator and then launhc the flyout and watch behavior.

Expected Results

Expect flyout to automatically fall into focus when launch or isopen, which would then allow keyboard focus and narrator focus to work.

CLI version

11.3.7

Environment

System:
  OS: Windows 11 10.0.26100
  CPU: (16) x64 Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz
  Memory: 12.90 GB / 63.95 GB
Binaries:
  Node:
    version: 18.17.1
    path: C:\Program Files\nodejs\node.EXE
  Yarn:
    version: 1.22.19
    path: C:\ProgramData\global-npm\yarn.CMD
  npm:
    version: 9.6.7
    path: C:\Program Files\nodejs\npm.CMD
  Watchman: Not Found
SDKs:
  Android SDK: Not Found
  Windows SDK:
    AllowDevelopmentWithoutDevLicense: Enabled
    Versions:
      - 10.0.18362.0
      - 10.0.19041.0
      - 10.0.22621.0
IDEs:
  Android Studio: Not Found
  Visual Studio:
    - 17.12.35506.116 (Visual Studio Enterprise 2022)
Languages:
  Java: Not Found
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react: Not Found
  react-native: Not Found
  react-native-windows: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: Not found
  newArchEnabled: Not found
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Community Modules

No response

Target Platform Version

None

Target Device(s)

Desktop

Visual Studio Version

Visual Studio 2022

Build Configuration

None

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

No response

@sgmuiruri sgmuiruri added the bug label Dec 10, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Dec 10, 2024
@jonthysell jonthysell added Area: Flyout Area: Paper Parity: Windows RNW does not look or behave like non-RNW Windows and removed Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) labels Dec 12, 2024
@jonthysell jonthysell added this to the Next milestone Dec 12, 2024
@microsoft-github-policy-service microsoft-github-policy-service bot added the Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper label Dec 12, 2024
@Yajur-Grover
Copy link
Contributor

So I have tried repro'ing this issue, and I am hitting the same behavior when clicking on the Flyout with Narrator on - focus does not move to the flyout. However, when using keyboard input, the focus works by moving onto the flyout. Looking at the email thread for the same issue, looks like this is the same behavior you are experiencing? Do you have an example of the way you tried to use ref management to fix the issue?

@Yajur-Grover
Copy link
Contributor

I faced a similar issue with Popup and used refs to allow focus to shift automatically using both keyboard and mouse input: microsoft/react-native-gallery#319. Can you try using refs like this to resolve the issue?

@chrisglein chrisglein added the Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) label Dec 18, 2024
@sgmuiruri
Copy link
Author

sgmuiruri commented Dec 19, 2024

@Yajur-Grover, have you had success with that approach? I will take a deeper look to understand what the expectation is here.

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Attention 👋 An issue that had been tagged "Needs: Author Feedback" has received activity (label applied by bot) and removed Needs: Author Feedback The issue/PR needs activity from its author (label drives bot activity) labels Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Flyout Area: Paper bug Needs: Attention 👋 An issue that had been tagged "Needs: Author Feedback" has received activity (label applied by bot) Old Architecture Broad category for issues that apply to the RN "old" architecture of Cxx Modules + Paper Parity: Windows RNW does not look or behave like non-RNW Windows
Projects
None yet
Development

No branches or pull requests

4 participants