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

[Bug]: Dialog focus is broken after a nested Popover is opened and closed #33230

Open
2 tasks done
tvandalovsky opened this issue Nov 7, 2024 · 3 comments
Open
2 tasks done

Comments

@tvandalovsky
Copy link

tvandalovsky commented Nov 7, 2024

Component

Dialog

Package version

9.55.1

React version

18.3.12

Environment

System:
OS: Linux 5.0 undefined
CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
Memory: 0 Bytes / 0 Bytes
Shell: 1.0 - /bin/jsh
npmPackages:
@fluentui/react-components: ^9.55.1 => 9.55.1
@types/react: ^18.3.12 => 18.3.12
@types/react-dom: ^18.3.1 => 18.3.1
react: ^18.3.1 => 18.3.1
react-dom: ^18.3.1 => 18.3.1

Current Behavior

Unable to re-focus the Dialog SearchBox after the nested Popover is opened and closed (by clicking the mouse outside of the Popover).

Expected Behavior

Be able to re-focus and engage with the Dialog SearchBox after the nested Popover is opened and closed (by clicking the mouse outside of the Popover).

Reproduction

https://stackblitz.com/edit/vitejs-vite-dvqcpm?file=src%2FApp.tsx

Steps to reproduce

  1. Press the "Open" button to open the Dialog
  2. Press the "Popover" button to open the Popover
  3. Click empty space outside of the Popover
  4. Try to click into the Dialog SearchBox
  5. Observe that the Dialog SearchBox cannot be focused, and the focus is "stuck" on the Dialog Open button.

Are you reporting an Accessibility issue?

yes

Suggested severity

Urgent - No workaround and Products/sites are affected

Products/sites affected

PowerScribe One

Are you willing to submit a PR to fix?

yes

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.
@miroslavstastny
Copy link
Member

miroslavstastny commented Nov 11, 2024

Perhaps the same root cause as #33017.
fyi @mshoho

@bsunderhus
Copy link
Contributor

Here's a reproduction using react@17 and adding a FluentProvider wrapper to ensure proper styling https://stackblitz.com/edit/vitejs-vite-ztxunm?file=src%2FApp.tsx

@bsunderhus
Copy link
Contributor

I believe this issue and #33017 are the same. In this example provided by @tvandalovsky the Popover is with trapFocus attribute and the logic behind PopoverSurface includes aria-modal=true and role=dialog to the popover once trapFocus is true.

role: trapFocus ? 'dialog' : 'group',
'aria-modal': trapFocus ? true : undefined,

This is only reproducible with trapFocus attribute on. This leads me to believe it might be related to some modalizer logic on tabster. Even after adding inline to the Popover the problem remains also 🤔

@mshoho is this a known issue on tabster?

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

No branches or pull requests

4 participants