Skip to content

FIX Restore focus after closing Feedback dialog#2223

Merged
romanlutz merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-restore-feedback-focus
Jul 20, 2026
Merged

FIX Restore focus after closing Feedback dialog#2223
romanlutz merged 2 commits into
microsoft:mainfrom
romanlutz:romanlutz-restore-feedback-focus

Conversation

@romanlutz

Copy link
Copy Markdown
Contributor

Description

Closing the Feedback dialog previously left keyboard focus on the document body, forcing keyboard and assistive-technology users to restart navigation. This change registers the persistent Feedback navigation button as Fluent UI's focus-restoration target so focus returns to the invoking control whenever the dialog closes.

Tests and Documentation

  • Added Playwright coverage for Escape and Cancel dismissal paths.
  • Ran the focused Navigation unit tests.
  • Ran the frontend TypeScript type-check and ESLint on the changed files.
  • Ran the focused Chromium accessibility regressions.
  • Documentation: N/A.
  • JupyText: N/A.

Copilot AI added 2 commits July 17, 2026 15:47
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 588eae27-1d87-453a-8bb8-0c11484ca6d1
@jsong468 jsong468 self-assigned this Jul 20, 2026
@romanlutz
romanlutz added this pull request to the merge queue Jul 20, 2026
Merged via the queue into microsoft:main with commit 5a4d899 Jul 20, 2026
47 checks passed
@romanlutz
romanlutz deleted the romanlutz-restore-feedback-focus branch July 20, 2026 22:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants