-
Notifications
You must be signed in to change notification settings - Fork 27.4k
Conversation
9af7ab5
to
bf8718e
Compare
use certain directives. To enable `ngAria`, just require the module into your application and | ||
the code will hook into your ng-show/ng-hide, input, textarea, button, select and | ||
ng-required directives and add the appropriate ARIA states and properties. | ||
The goal of ngAria is to improve Angular's default accessibility by enabling common [ARIA](http://www.w3.org/TR/wai-aria/) attributes that convey state or semantic information for assistive technologies used by persons with disabilities. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we wrap these lines at 100cols so it's a bit easier to review?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done!
bf8718e
to
82c5cac
Compare
These looks pretty good --- I would still like to see more content in the API docs section for ngAria, especially code examples, and a specific list of each core directive which is "overloaded", and in what capacity. But the guide looks great. Note you'll want to add it to the TOC on https://github.com/angular/angular.js/blob/master/docs/content/guide/index.ngdoc --- and it would be really cool if we had a link to it on the docs index page to highlight it, too. |
@caitp great suggestions. Isn't that TOC the docs index page or is there another one? I have added a link to the guide index page that you linked. I can do a pass on the API documentation but it may have to come after this PR. |
82c5cac
to
bbfc397
Compare
The guide index page should be fine --- but the other index page is https://github.com/angular/angular.js/blob/master/docs/content/api/index.ngdoc -- there's a little overview of each module, but ngAria is missing. That stuff can wait for another PR, this is good so I'd like to land it. |
bbfc397
to
e200515
Compare
LGTM |
LGTM - Thanks @marcysutton |
@caitp should |
I believe that page is generated automatically --- @petebacondarwin am I wrong about that? I think all you need is doc-annotated comments on the $ariaProvider. |
That would make much more sense. No human should be generating these HTML tables, let alone reading them. :) |
@marcysutton: |
Hi @gkalpak, I'm not sure I follow. The |
@marcysutton: Now, I am not sure I follow. The In my comment, I was referring to #9930 (comment):
Expanding on my previous comment (for the sake of clarity):
|
@gkalpak gotcha! That makes sense, thank you for the clarification. |
To help explain what the ngAria module can and can't do, I've updated the Developer Guide to include more in-depth information, code examples and accessibility resources. This is a companion piece to a blog post on the official AngularJS blog due out shortly.
Related to #9925