Skip to content

fix: restore menu list keyboard navigation after reconnect - #36871

Open
aseeb shibin (asxxb) wants to merge 2 commits into
microsoft:masterfrom
asxxb:fix/menu-list-focusgroup-reconnect
Open

aseeb shibin (asxxb) wants to merge 2 commits into
microsoft:masterfrom
asxxb:fix/menu-list-focusgroup-reconnect

Conversation

@asxxb

Copy link
Copy Markdown

Previous Behavior

When a fluent-menu-list element was removed from the DOM and reinserted, its FocusGroup instance remained referenced after being disconnected. As a result, arrow-key navigation could stop working in Firefox and Safari.

New Behavior

Clear the existing FocusGroup reference when MenuList disconnects, allowing a new instance to be created when the element reconnects.

Add a regression test to verify that arrow-key navigation continues working after the menu list is moved in the DOM.

Related Issue(s)

@asxxb
aseeb shibin (asxxb) requested a review from a team as a code owner October 12, 2026 23:23
@asxxb

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

@marchbox

Copy link
Copy Markdown
Contributor

aseeb shibin (@asxxb) Please run yarn change in root to add a change file

@github-actions

Copy link
Copy Markdown

📊 Bundle size report

✅ No changes found

@github-actions

Copy link
Copy Markdown

Pull request demo site: URL

Comment thread packages/web-components/src/menu-list/menu-list.spec.ts
@asxxb

Copy link
Copy Markdown
Author

aseeb shibin (aseeb shibin (@asxxb)) Please run yarn change in root to add a change file

Hi! I've added the required Beachball change file and pushed the update. The check-packages workflow is currently awaiting maintainer approval. Could you please approve the workflow run when convenient? Thank you!

@asxxb
aseeb shibin (asxxb) force-pushed the fix/menu-list-focusgroup-reconnect branch from c970609 to 5a54f76 Compare October 24, 2026 19:48

This branch has not been deployed

No deployments
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.

@fluentui/web-components: fluent-menu-list loses its arrow keys after it is moved in the DOM (Firefox, Safari)

2 participants