Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement all role & aria properties in RFC #47

Merged
merged 1 commit into from
Feb 10, 2016

Conversation

cibernox
Copy link
Owner

See cibernox/ember-power-select#293 (comment)

  • role=button
  • aria-haspopup=true tells AT to treat the trigger as a host for conditional content
  • aria-expanded=true/false tells AT that conditional content is expanded
  • aria-pressed=true/false tells AT that the button is in a pressed or 'true' state (technically different from whether the conditional content is expanded, but in practice usually the same).
  • aria-controls=<id of dropdown> is all we need since there is no requirement that the trigger element own or have a direct DOM relationship to the dropdown

See cibernox/ember-power-select#293 (comment)

 - [x] `role=button`
 - [x] `aria-haspopup=true` tells AT to treat the trigger as a host for conditional content
 - [x] `aria-expanded=true/false` tells AT that conditional content is expanded
 - [x] `aria-pressed=true/false` tells AT that the `button` is in a pressed or 'true' state (technically different from whether the conditional content is expanded, but in practice usually the same).
 - [x] `aria-controls=<id of dropdown>` is all we need since there is no requirement that the trigger element own or have a direct DOM relationship to the dropdown
cibernox added a commit that referenced this pull request Feb 10, 2016
…ibutes

Implement all role & aria properties in RFC
@cibernox cibernox merged commit 987d498 into master Feb 10, 2016
@cibernox cibernox deleted the implement_all_role-and-aria-attributes branch February 10, 2016 13:21
@jmacqueen
Copy link
Contributor

Woo! 👍 👍 👍

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