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

docs($interval): improve fn description #15503

Closed
wants to merge 1 commit into from
Closed

docs($interval): improve fn description #15503

wants to merge 1 commit into from

Conversation

jannickfahlbusch
Copy link
Contributor

@jannickfahlbusch jannickfahlbusch commented Dec 13, 2016

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
docs update:

If no arguments were specified, angular passes the current iteration to the funtion (here)

Does this PR introduce a breaking change?
No

Please check if the PR fulfills these requirements

@@ -33,7 +33,8 @@ function $IntervalProvider() {
* appropriate moment. See the example below for more details on how and when to do this.
* </div>
*
* @param {function()} fn A function that should be called repeatedly.
* @param {function()} fn A function that should be called repeatedly. If no arguments are passed,
* the function is called with the current iteration.
Copy link
Member

Choose a reason for hiding this comment

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

Can you change it to something like:

If not additional arguments are passed (see below), the function is called with the current iteration count.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, cee8801

If no additional arguments are passed, the function is called with the current iteration count.
gkalpak pushed a commit that referenced this pull request Dec 13, 2016
If no additional arguments are passed, the function is called with the current iteration count.

Closes #15503
@gkalpak gkalpak closed this in e9a4de0 Dec 13, 2016
gkalpak pushed a commit that referenced this pull request Dec 13, 2016
If no additional arguments are passed, the function is called with the current iteration count.

Closes #15503
@jannickfahlbusch jannickfahlbusch deleted the patch-1 branch February 3, 2017 11:46
ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
If no additional arguments are passed, the function is called with the current iteration count.

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

Successfully merging this pull request may close these issues.

3 participants