Skip to content

fix: Improve ARIA fallback generation for HTML-based menu items - #9827

Merged
gonfunko merged 2 commits into
v13from
menu-readout
May 7, 2026
Merged

fix: Improve ARIA fallback generation for HTML-based menu items#9827
gonfunko merged 2 commits into
v13from
menu-readout

Conversation

@gonfunko

@gonfunko gonfunko commented May 7, 2026

Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes @microbit-robert's comment in #9785

Proposed Changes

This PR improves the fallback ARIA label generation of HTML-based contextual menu items. Previously, menu items that didn't have an explicit ARIA label set were wrapped in String(), which produced [object HTMLDivElement] or the like. Now, when the menu item is HTML-based, its textContent field is used.

@gonfunko
gonfunko requested a review from a team as a code owner May 7, 2026 16:11
@gonfunko
gonfunko requested a review from maribethb May 7, 2026 16:11
@github-actions github-actions Bot added the PR: fix Fixes a bug label May 7, 2026
Comment thread packages/blockly/core/menuitem.ts Outdated
@gonfunko
gonfunko merged commit ce86628 into v13 May 7, 2026
4 checks passed
@gonfunko
gonfunko deleted the menu-readout branch May 7, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: fix Fixes a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants