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

docs($animate): detail options object #14700

Merged
merged 1 commit into from
Jun 3, 2016

Conversation

wesleycho
Copy link
Contributor

  • Document options object used in various $animate methods

This should address #12645.

* - **from** - `{Object}` - CSS properties & values from beginning of animation
* - **removeClass** - `{string}` - space-separated CSS classes to remove from element
* - **to** - `{Object}` - CSS properties & values at end of animation
*
Copy link
Member

Choose a reason for hiding this comment

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

Could you also document the options argument of other methods (e.g. move()) ?

@gkalpak
Copy link
Member

gkalpak commented Jun 1, 2016

This should indeed fix #12645. Feel free to add Fixes #12645 in the commit message.

@wesleycho wesleycho force-pushed the docs/animate-options branch from b5b969d to 55d0c88 Compare June 1, 2016 13:20
* The object has the following properties:
*
* - **addClass** - `{string}` - space-separated CSS classes to add to element
* - **from** - `{Object}` - CSS properties & values from beginning of animation
Copy link
Contributor

Choose a reason for hiding this comment

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

"from the beginning" -> "at the beginning"

- Document options object used in various $animate methods

Fixes angular#12645
@wesleycho wesleycho force-pushed the docs/animate-options branch from 55d0c88 to 9dc3fe4 Compare June 2, 2016 13:39
@Narretz Narretz merged commit 146dad7 into angular:master Jun 3, 2016
Narretz pushed a commit that referenced this pull request Jun 3, 2016
- Document options object used in various $animate methods

Fixes #12645
PR: #14700
gkalpak pushed a commit to gkalpak/angular.js that referenced this pull request Jun 3, 2016
- Document options object used in various $animate methods

Fixes angular#12645
PR: angular#14700
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