Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

ngDisabled should also set aria-disabled #279

Closed
marcysutton opened this issue Sep 17, 2014 · 3 comments
Closed

ngDisabled should also set aria-disabled #279

marcysutton opened this issue Sep 17, 2014 · 3 comments
Labels
a11y This issue is related to accessibility type: enhancement

Comments

@marcysutton
Copy link
Contributor

Because of Angular.js Material Design's use of element directives, aria-disabled must be set alongside disabled to communicate a control's state to assistive technologies. disabled is only valid on a subset of HTML elements or their prototypes (using Web Components), including input, button, select, and more. More on disabled elements: http://www.w3.org/TR/html5/disabled-elements.html

This affects material-checkbox, material-button, material-radio, material-tabs, material-slider and any component that can be disabled.

@marcysutton marcysutton added type: enhancement a11y This issue is related to accessibility labels Sep 17, 2014
@ThomasBurleson
Copy link
Contributor

@marcysutton - great issue here!

@marcysutton
Copy link
Contributor Author

I'm actually seeing incorrect aria-disabled values when inputs are disabled:
Material slider in Chrome devtools shows incorrect attributes

Material-slider has disabled and aria-disabled="false", which is a mismatch. Could be related to #272.

@marcysutton
Copy link
Contributor Author

Closed with ngAria added to project 96c0095

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y This issue is related to accessibility type: enhancement
Projects
None yet
Development

No branches or pull requests

2 participants