Skip to content

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

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

fix: [a11y] fire AXMenuOpened event when ARIA menu is added to DOM#50504
jkleinsc merged 5 commits into42-x-yfrom
trop/42-x-y-bp-fix-a11y-fire-axmenuopened-event-when-aria-menu-is-added-to-dom-1774474446615

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 42-x-y backport This is a backport PR semver/patch backwards-compatible bug fixes labels Mar 25, 2026
@jkleinsc jkleinsc merged commit f90dd8d into 42-x-y Mar 26, 2026
104 of 106 checks passed
@jkleinsc jkleinsc deleted the trop/42-x-y-bp-fix-a11y-fire-axmenuopened-event-when-aria-menu-is-added-to-dom-1774474446615 branch March 26, 2026 15:45
@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

42-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