Skip to content

fix(settings): Close member dropdown before invite modal - #121005

Open
scttcper wants to merge 1 commit into
masterfrom
scttcper/member-invite-focus
Open

fix(settings): Close member dropdown before invite modal#121005
scttcper wants to merge 1 commit into
masterfrom
scttcper/member-invite-focus

Conversation

@scttcper

@scttcper scttcper commented Jul 31, 2026

Copy link
Copy Markdown
Member

Opening Invite Member from the Add Member menu left the menu's contained focus scope active while the modal created a second focus trap. They could bounce focus between each other and freeze the page.

Close the menu before opening the modal and cover the behavior in the team members test.

steps to reproduce:

  • go to team settings page and open "Add member" dropdown
  • click invite member

fixes JAVASCRIPT-3B6A

The member dropdown kept its contained focus scope active while the invite modal opened its own focus trap. The two scopes could fight over focus and freeze the page.

Close the dropdown first and assert that it is removed when the invite action runs.

Co-Authored-By: Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jul 31, 2026
@scttcper
scttcper marked this pull request as ready for review July 31, 2026 20:43
@scttcper
scttcper requested a review from a team as a code owner July 31, 2026 20:43
@scttcper
scttcper requested a review from a team July 31, 2026 20:45
scttcper added a commit that referenced this pull request Jul 31, 2026
Opening Invite Member from the assignee selector left the menu's focus
scope active while the global modal created a second focus trap. That
can bounce focus between both overlays and freeze the page.

Close the assignee menu before launching the modal, matching #121005.

Co-authored-by: Codex <noreply@openai.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant