FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fixed animation value sync issues by tzraikov · Pull Request #2266 · NativeScript/NativeScript · GitHub

Fixed animation value sync issues - #2266

Merged
tzraikov merged 1 commit into
masterfrom
raikov/fix-animationsync
Jun 17, 2016
Merged

tzraikov merged 1 commit into
masterfrom
raikov/fix-animationsync

Conversation

tzraikov commented Jun 8, 2016

Copy link
Copy Markdown
Contributor

Fixes the following scenario (existing in iOS):

  1. Execute an animation (e.g. translate 0, 100)
  2. Reset transformX, transformY values with code
  3. Animate again (translate 0, 100) and the animation will look wrong because style property values will be out of sync compared to the View properties.

ns-bot added the cla: yes label Jun 8, 2016
tzraikov self-assigned this Jun 8, 2016

Copy link
Copy Markdown
Contributor

👍

tzraikov force-pushed the raikov/fix-animationsync branch 3 times, most recently from 2275d3d to a03baf0 Compare June 9, 2016 13:40
vchimev closed this Jun 9, 2016
vchimev reopened this Jun 9, 2016
this._reject(new Error("Animation cancelled."));
}

private _resetAnimationValues(view: view.View, animation: Object) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Why do we need to reset animation values?

tzraikov force-pushed the raikov/fix-animationsync branch 2 times, most recently from f15f86a to 9566ecb Compare June 13, 2016 16:12
tzraikov force-pushed the raikov/fix-animationsync branch from 79c2fb9 to d810a77 Compare June 14, 2016 08:31

vchimev commented Jun 14, 2016

Copy link
Copy Markdown
Contributor

👍

Copy link
Copy Markdown
Contributor Author

run ci

tzraikov merged commit 6275c71 into master Jun 17, 2016
tzraikov deleted the raikov/fix-animationsync branch June 17, 2016 14:54
hshristov added this to the 2.1 milestone Jul 1, 2016

lock Bot commented Aug 28, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 28, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL