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

fix(rootScope.js): edit $watch parameter comment #8322

Closed
wants to merge 1 commit into from
Closed

fix(rootScope.js): edit $watch parameter comment #8322

wants to merge 1 commit into from

Conversation

dbwhddn10
Copy link
Contributor

i think $watch listener params comment is wrong ( string value is not correct)

i think $watch listener params is same as $watchgroup listener and only function value

https://github.com/angular/angular.js/blob/master/src/ng/rootScope.js#L349-351

i think $watch listener params comment is wrong ( string value is not correct)

i think $watch listener params is same as $watchgroup listener
@mary-poppins
Copy link

Thanks for the PR! Please check the items below to help us merge this faster. See the contributing docs for more information.

  • Uses the issue template (#8322)

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!

@dbwhddn10 dbwhddn10 mentioned this pull request Jul 24, 2014
@cmmartin
Copy link

You can pass a string (an angular expression) to be executed as a $watch listener.

For example...
$scope.$watch('count1', 'count2 = count1');

http://jsfiddle.net/HB7LU/5156/

@caitp
Copy link
Contributor

caitp commented Jul 24, 2014

@cmmartin #8190

@cmmartin
Copy link

thanks :)

@petebacondarwin
Copy link
Contributor

@dbwhddn10 - thanks for this. The description is not quite accurate but I have modified it and merged.

@dbwhddn10 dbwhddn10 deleted the patch-1 branch July 25, 2014 21:53
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.

7 participants