Skip to content

Improve screen reader behaviour for area overlay#11152

Open
microbit-robert wants to merge 1 commit intomicrosoft:masterfrom
microbit-robert:area-menu-screen-reader
Open

Improve screen reader behaviour for area overlay#11152
microbit-robert wants to merge 1 commit intomicrosoft:masterfrom
microbit-robert:area-menu-screen-reader

Conversation

@microbit-robert
Copy link
Contributor

The current area overlay menu is keyboard accessible but the screen reader experience is not so good. This PR makes the following changes to improve the screen reader experience:

  • Add roles of menu and menuitem
  • Add aria-label for menu
  • Add arrow navigation and focus the first menu item by default
  • Align the area overlay menu item names with the accessibility documentation

Add role of menu and menuitem.
Add aria-label for menu.
Add arrow navigation and focus the first menu item by default.
Align area overlay menu item names with accessibility docs.
id: "editor",
ariaLabel: lf("Editor"),
id: "workspace",
ariaLabel: lf("Workspace"),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this only be "Workspace" for the Blocks case, and "Editor" for the JavaScript/Python case?

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.

1 participant