Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Add in-depth guide for ngAria #9930

Closed
wants to merge 1 commit into from

Conversation

marcysutton
Copy link
Contributor

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

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.
Copy link
Contributor

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?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

@caitp
Copy link
Contributor

caitp commented Nov 5, 2014

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.

@marcysutton
Copy link
Contributor Author

@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.

@caitp
Copy link
Contributor

caitp commented Nov 5, 2014

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.

@btford btford self-assigned this Nov 6, 2014
@btford
Copy link
Contributor

btford commented Nov 6, 2014

LGTM

@btford btford added this to the 1.3.2 milestone Nov 6, 2014
@petebacondarwin
Copy link
Contributor

LGTM - Thanks @marcysutton

@caitp caitp closed this in c6909ed Nov 6, 2014
@marcysutton
Copy link
Contributor Author

@caitp should $ariaProvider be added to the Provider components doc? https://docs.angularjs.org/api/ng/provider

@caitp
Copy link
Contributor

caitp commented Nov 10, 2014

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.

@marcysutton
Copy link
Contributor Author

That would make much more sense. No human should be generating these HTML tables, let alone reading them. :)

@gkalpak
Copy link
Member

gkalpak commented Nov 11, 2014

@marcysutton:
The doc you link to is for providers in the ng module (and this is not where $ariaProvider belongs).
The corresponding section for the ngAria module is https://docs.angularjs.org/api/ngAria/provider (and $ariaProvider is indeed listed there).

@marcysutton
Copy link
Contributor Author

Hi @gkalpak, I'm not sure I follow. The config link on L128 of this Developer Guide gets the user to currently-available information without having to follow a trail of links around--is there a convention for docs contributors that I'm missing? The API documentation will be updated later this week, providing more content that can be linked to directly from this guide.

@gkalpak
Copy link
Member

gkalpak commented Nov 11, 2014

@marcysutton: Now, I am not sure I follow. The config link seems fine and should take people to https://docs.angularjs.org/api/ngAria/provider/$ariaProvider (the docs on $ariaProvider), as expected.
This is fine.


In my comment, I was referring to #9930 (comment):

@caitp should $ariaProvider be added to the Provider components doc? https://docs.angularjs.org/api/ng/provider

Expanding on my previous comment (for the sake of clarity):

$ariaProvider should not be added to the Provider components doc at https://docs.angularjs.org/api/ng/provider (<--- this is the link you used in your question), because it refers to the Providers in the ng module (core).

$ariaProvider belongs to the ngAria module (as you know better than me 😸) and therefore it should appear in the Provider components doc for the ngAria module (at https://docs.angularjs.org/api/ngAria/provider).
And indeed it already appears there, since it is correctly ngdoc-annotated (as provider of the ngAria module). Therefore, you don't need to do anything manually.

I hope this clears it up. Let me know if it doesn't 😃

@marcysutton
Copy link
Contributor Author

@gkalpak gotcha! That makes sense, thank you for the clarification.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants