Skip to content

Add aria-description for menu keyboard shortcuts#539

Merged
sjdemartini merged 1 commit intomainfrom
steven/a11y-friendly-keyboard-shortcut-descriptions
Mar 22, 2026
Merged

Add aria-description for menu keyboard shortcuts#539
sjdemartini merged 1 commit intomainfrom
steven/a11y-friendly-keyboard-shortcut-descriptions

Conversation

@sjdemartini
Copy link
Owner

Alternative approach to #509 that separates from the the standard (and more important) aria-label for the element. And extends to using it in other places besides just the menu button.

@sjdemartini sjdemartini force-pushed the steven/a11y-friendly-keyboard-shortcut-descriptions branch from 9820325 to 46ab78e Compare March 22, 2026 03:40
@sjdemartini sjdemartini changed the title Add aria-description for keyboard shortcuts Add aria-description for menu keyboard shortcuts Mar 22, 2026
@sjdemartini sjdemartini merged commit 7cd0922 into main Mar 22, 2026
10 checks passed
@sjdemartini sjdemartini deleted the steven/a11y-friendly-keyboard-shortcut-descriptions branch March 22, 2026 03:46
@shysteph
Copy link
Contributor

I am not sure that is a valid use of aria-description. Description should be a longer worded description of what the the visual element is and is not fully supported yet unlike aria-describedby. I think both should be in the label or the button element should be aria-describedby the id of the tooltip popup (this is probably most correct and then I don't think it would even need a label. This is also preferred for tooltips.

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.

2 participants