Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

animationStart doubly triggered #23

Closed
sculove opened this issue Dec 21, 2015 · 3 comments
Closed

animationStart doubly triggered #23

sculove opened this issue Dec 21, 2015 · 3 comments
Assignees
@sculove
Labels
bug movableCoord
Milestone
1.2.0

Comments

@sculove
Copy link
Contributor

sculove commented Dec 21, 2015

as-is

hold
release
animationStart
animationStart
animationEnd

to-be

hold
release
animationStart (isBounce : false)
animationEnd (isBounce : false)
animationStart (isBounce : true)
animationEnd (isBounce : true)
@mixed mixed added this to the 1.1.0 milestone Jan 11, 2016
@sculove
Copy link
Contributor Author

sculove commented Jan 13, 2016

we should think together with #5.

@sculove sculove removed this from the 1.1.0 milestone Jan 13, 2016
@sculove sculove mentioned this issue Mar 14, 2016
Closed
@sculove
Copy link
Contributor Author

sculove commented Mar 23, 2016

AS-IS

case 1 : tap

  • hold => release

case 2 : down and move, up

  • hold => change ... => release

case 3 : down and move, up, animation

  • hold => change ... => release => animationStart => change ... => animationEnd

case 4 : down and move, up, animation, bounce

  • hold => change ... => relase => animationStart => change ... => animationStart => change ... => animationEnd

case 5 : down and move, up, animation (interrupt)

  • hold => change ... => release => animationStart => change ... => hold => release

case 6 : down and move, up, animation, bounce (interrupt)

  • hold => change ... => release => animationStart => change ... => animationStart => change ... => hold => release

@sculove sculove modified the milestone: 1.2.0 Mar 23, 2016
@sculove
Copy link
Contributor Author

sculove commented Mar 24, 2016

TO-BE

case 4 : down and move, up, animation, bounce

  • hold => change ... => relase => animationStart => change ... => animationStart (remove)=> change ... => animationEnd

case 5 : down and move, up, animation (interrupt)

  • hold => change ... => release => animationStart => change ... => animationEnd (add) => hold => release

case 6 : down and move, up, animation, bounce (interrupt)

  • hold => change ... => release => animationStart => change ... => animationStart (remove) => change ... => animationEnd (add) => hold => release

@sculove sculove mentioned this issue Mar 28, 2016
Closed
@sculove sculove closed this as completed Apr 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sculove sculove

Labels
bug movableCoord
Projects
None yet
Milestone
1.2.0
Development

No branches or pull requests
2 participants
@mixed @sculove