Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

docs(forms): copyedits #3149

Merged
merged 2 commits into from
Jan 26, 2017
Merged

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jan 26, 2017

Copyedits from syncing with Dart version of page.

  • Copyedits, some from Kathy (from an early revision of the forms page).
    • Also, there were some missing closing code ticks (`), and missing & extraneous :marked.
  • Removed side note detailing ngModel; we're now referring reader to ngModel template syntax section instead.

cc @kwalrath @kapunahelewong

- Removed side note detailing ngModel; now referring reader to
template-syntax page instead.
Copy link
Contributor

@kwalrath kwalrath left a comment

Choose a reason for hiding this comment

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

I didn't check these changes carefully, but they seem reasonable. (I don't remember which changes came from who, but I was surprised that improvements we had in the Dart version weren't already in this version.)

@kapunahelewong could you do a more thorough review?


:marked
It's an anemic model with few requirements and no behavior. Perfect for our demo.

The TypeScript compiler generates a public field for each `public` constructor parameter and
assigns the parameter’s value to that field automatically when we create new heroes.

The `alterEgo` is optional and the constructor lets us omit it; note the (?) in `alterEgo?`.
The `alterEgo` is optional, so the constructor will let us omit it; note the (?) in `alterEgo?`.
Copy link
Contributor

Choose a reason for hiding this comment

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

will let -> lets

(not a big deal, but strange that it's worse)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

- display validation errors to users and enable/disable form controls

- use [template reference variables](./template-syntax.html#ref-vars) for sharing information among HTML elements
- Build an Angular form with a component and template
Copy link
Contributor

Choose a reason for hiding this comment

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

According to the guidelines, we're supposed to put a period at the end of each list item, however, I have seen that not applied to TOCs, which I consider this to be. However, you'd be within the guidelines by having periods on each since it isn't labeled explicitly as a TOC. I'd probably put them but considering the context, it's arguable not to have them.

1. Add custom CSS to provide visual feedback.
1. Show and hide validation error messages.
1. Handle form submission with **ngSubmit**.
1. Disable the form’s submit button until the form is valid.
Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, I see you know what I said in my first comment.

@filipesilva filipesilva merged commit 806c658 into angular:master Jan 26, 2017
@kwalrath kwalrath deleted the chalin-forms-updates-0123 branch January 26, 2017 22:25
abdel-ships-it pushed a commit to abdel-ships-it/angular.io that referenced this pull request Feb 11, 2017
* - Copyedits, some from Kathy (from an early revision of the forms page).
- Removed side note detailing ngModel; now referring reader to
template-syntax page instead.

* post-review edits
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.

5 participants