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

docs(guide/Working With CSS): ng-touched/untouched #10302

Closed
wants to merge 1 commit into from

Conversation

philippd
Copy link
Contributor

@philippd philippd commented Dec 2, 2014

Add description for ng-touched and ng-untouched CSS classes.

Add description for ng-touched and ng-untouched CSS classes.
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@philippd
Copy link
Contributor Author

philippd commented Dec 2, 2014

CLA signed

@googlebot
Copy link

CLAs look good, thanks!

@googlebot googlebot added cla: yes and removed cla: no labels Dec 2, 2014
@@ -28,6 +28,11 @@ Angular sets these CSS classes. It is up to your application to provide useful s
- **Usage:** angular {@link ng.directive:input input} directive applies `ng-pristine` class
to a new input widget element which did not have user interaction. Once the user interacts with
the input widget the class is changed to `ng-dirty`.

* `ng-touched`, `ng-untouched`
- **Usage:** angular {@link ng.directive:input input} directive applies `ng-untouched` class
Copy link
Contributor

Choose a reason for hiding this comment

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

It's not really the input directive, it's ngModel --- Want to make that little adjustment? Otherwise looks okay.

@caitp caitp added this to the 1.3.6 milestone Dec 2, 2014
@caitp caitp self-assigned this Dec 2, 2014
@caitp caitp closed this in c8c9bbc Dec 2, 2014
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.

3 participants