-
Notifications
You must be signed in to change notification settings - Fork 27.4k
Updated README.md - Add additional introductory paragraph #15150
Updated README.md - Add additional introductory paragraph #15150
Conversation
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
1 similar comment
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed, please reply here (e.g.
|
Updated introductory paragraph to README for further clarification of how Angular works. |
CLAs look good, thanks! |
1 similar comment
CLAs look good, thanks! |
@@ -8,6 +8,8 @@ synchronizes data from your UI (view) with your JavaScript objects (model) throu | |||
binding. To help you structure your application better and make it easy to test, AngularJS teaches | |||
the browser how to do dependency injection and inversion of control. | |||
|
|||
HTML is also used to determine the execution of the app. Special attributes in the HTML determine which controllers to use for each element. These attributes determine "what" gets loaded, but not "how". This declarative approach greatly simplifies app development in a sort of WYSIWYG way. Rather than spending time on how the program flows and what should get loaded first, you simply define what you want and Angular will take care of the dependencies. |
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.
I feel this goes into a bit too much detail for this high-level description here.
Maybe incorporate it into the Interconnection with HTML at the root level
section below (or in it's own section).
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.
Also, please, wrap lines at 100 characters max 😃
@frankstepanski Can you incorporate the changes @gkalpak requested? Otherwise we'll have to close this PR. |
Sorry, missed this...will incorporate it his weekend. thanks. |
…oper-readme-contribution
Updated |
I tweaked it a bit and merged. Thx! |
Thanks! |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change?
Please check if the PR fulfills these requirements
Other information: