-
Notifications
You must be signed in to change notification settings - Fork 1k
Description
Description
Today, button and icon button only support native [disabled]. They should also support aria-disabled.
The chip component exposes an always-focusable attribute for this:
material-web/chips/internal/chip.ts
Line 46 in 7867674
| alwaysFocusable = false; |
We could copy that same API, or we could add a new aria-disabled attribute to button and icon button.
Pros for always-focusable
- Aligns with chip
Pros for aria-disabled
- Aligns with existing web APIs
- Aligns with other
aria-*APIs exposed on button (e.g.aria-label,aria-haspopup, etc)
I also noticed while looking at chip's implementation that it doesn't set aria-disabled on its inner button if it's disabled and always focusable. It probably should?
| ?disabled=${this.disabled && !this.alwaysFocusable} |
Metadata
Metadata
Assignees
Labels
No labels