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

New paragraph for ng:autobind #328

Closed
wants to merge 1 commit into from

Conversation

ctataryn
Copy link
Contributor

I felt the ng:autobind could use another paragraph explaining the need for ng:autobind. It was one of the first things that struck my curiosity when I looked at Angular: why do we need to specify ng:autobind at all? The paragraph simply explains that Angular doesn't want to assume you always want to process the entire document when its script source is loaded.

@IgorMinar
Copy link
Contributor

Thanks for the contribution.

I felt like there was too much redundancy in that doc, so I simplified your change a bit but preserved the main goal of your change - to explain why ng:autobind is needed at all.

committed as 64938a2

@IgorMinar IgorMinar closed this Apr 16, 2011
@ctataryn
Copy link
Contributor Author

Great Igor. I'll make more changes as I come across things. Do you prefer them as separate pull requests or can I batch things in one request? I'm not sure what's easier for you folks?

On 2011-04-16, at 4:38 PM, IgorMinarreply@reply.github.com wrote:

Thanks for the contribution.

I felt like there was too much redundancy in that doc, so I simplified your change a bit but preserved the main goal of your change - to explain why ng:autobind is needed at all.

committed as 64938a2

Reply to this email directly or view it on GitHub:
#328 (comment)

@IgorMinar
Copy link
Contributor

we are working on the docs as well, so it might be easier to get changes in smaller batches. e.g. one pull request with doc changes made to one file.

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.

2 participants