Skip to content

fix: [a11y] fire AXMenuOpened event when ARIA menu is added to DOM#50505

Merged
jkleinsc merged 5 commits into40-x-yfrom
trop/40-x-y-bp-fix-a11y-fire-axmenuopened-event-when-aria-menu-is-added-to-dom-1774474447238
Mar 26, 2026
Merged

fix: [a11y] fire AXMenuOpened event when ARIA menu is added to DOM#50505
jkleinsc merged 5 commits into40-x-yfrom
trop/40-x-y-bp-fix-a11y-fire-axmenuopened-event-when-aria-menu-is-added-to-dom-1774474447238

Conversation

@trop
Copy link
Copy Markdown
Contributor

@trop trop bot commented Mar 25, 2026

Backport of #50377

See that PR for details.

Notes: Fixed an accessibility issue where the AXMenuOpened event was not fired on menu creation.

trop bot and others added 4 commits March 25, 2026 21:34
…ed to the DOM

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
MENU_POPUP_END for deleted menus is already handled by
AXTreeManager::OnNodeWillBeDeleted, which
fires the event directly on the menu node before destruction.

Co-authored-by: Keeley Hammond <khammond@slack-corp.com>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
Co-authored-by: Keeley Hammond <vertedinde@electronjs.org>
@trop trop bot requested a review from a team as a code owner March 25, 2026 21:34
@trop trop bot requested review from a team and VerteDinde March 25, 2026 21:34
@trop trop bot added 40-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Mar 25, 2026
@jkleinsc jkleinsc merged commit 36c88a4 into 40-x-y Mar 26, 2026
59 checks passed
@jkleinsc jkleinsc deleted the trop/40-x-y-bp-fix-a11y-fire-axmenuopened-event-when-aria-menu-is-added-to-dom-1774474447238 branch March 26, 2026 13:18
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Mar 26, 2026

Release Notes Persisted

Fixed an accessibility issue where the AXMenuOpened event was not fired on menu creation.

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

Labels

40-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants