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

Update dev_guide.mvc.understanding_controller.ngdoc #4373

Closed
wants to merge 1 commit into from
Closed

Update dev_guide.mvc.understanding_controller.ngdoc #4373

wants to merge 1 commit into from

Conversation

felixcriv
Copy link
Contributor

Just added .controller syntax to the examples. Feel free to accept this changes. I just feel that for new learners (as me) is more comfortable the controller syntax.

Just added `.controller` syntax to the examples.
@mary-poppins
Copy link

Thanks for the PR!

  • Contributor signed CLA now or in the past
    • If you just signed, leave a comment here with your real name
  • PR's commit messages follow the commit message format

If you need to make changes to your pull request, you can update the commit with git commit --amend.
Then, update the pull request with git push -f.

Thanks again for your help!

petebacondarwin added a commit that referenced this pull request Oct 11, 2013
Remove mention of global controller functions
Convert larger examples to runnable demos
Remove mention of pre-1.0 controllers, in particular discussion of
controller inheritance.

TODO: Probably could do with updating to explain the "controller as" syntax
at some point.

Closes: #4373
petebacondarwin added a commit that referenced this pull request Oct 11, 2013
Remove mention of global controller functions
Convert larger examples to runnable demos
Remove mention of pre-1.0 controllers, in particular discussion of
controller inheritance.

TODO: Probably could do with updating to explain the "controller as" syntax
at some point.

Closes: #4373
@petebacondarwin
Copy link
Contributor

Landed as e86aaa9 and added some further improvements as 10bae7b
Thanks

jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
Remove mention of global controller functions
Convert larger examples to runnable demos
Remove mention of pre-1.0 controllers, in particular discussion of
controller inheritance.

TODO: Probably could do with updating to explain the "controller as" syntax
at some point.

Closes: angular#4373
jamesdaily pushed a commit to jamesdaily/angular.js that referenced this pull request Jan 27, 2014
Remove mention of global controller functions
Convert larger examples to runnable demos
Remove mention of pre-1.0 controllers, in particular discussion of
controller inheritance.

TODO: Probably could do with updating to explain the "controller as" syntax
at some point.

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

Successfully merging this pull request may close these issues.

3 participants