Skip to content
This repository has been archived by the owner on May 29, 2019. It is now read-only.

feat(progressbar): add aria-labelledby support #4350

Closed

Conversation

wesleycho
Copy link
Contributor

  • Add aria-labelledby support for accessibility

This resolves #4247.

@@ -81,6 +81,7 @@ describe('progressbar directive', function() {
expect(bar.attr('aria-valuemax')).toBe('100');
expect(bar.attr('aria-valuenow')).toBe('22');
expect(bar.attr('aria-valuetext')).toBe('22%');
expect(bar.attr('aria-labelledby')).toBe('foo');
Copy link
Contributor

Choose a reason for hiding this comment

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

do we want a test case for when the title attribute is omitted? that the title is progressbar.

- Add `aria-labelledby` support for accessibility
@wesleycho
Copy link
Contributor Author

Updated as requested

@icfantv
Copy link
Contributor

icfantv commented Sep 9, 2015

👍

@wesleycho wesleycho closed this in e6f3b87 Sep 9, 2015
@wesleycho wesleycho deleted the feat/progress-accessibility branch September 9, 2015 07:55
jasonaden pushed a commit to deskfed/bootstrap that referenced this pull request Jan 8, 2016
- Add `aria-labelledby` support for accessibility

Closes angular-ui#4350
Resolves angular-ui#4347
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[a11y] aria label progressbar compliance
2 participants