transitionEnd should not be applied when new animations caused the animation to stop

[chuganzy]

This PR aims to fix an issue where transitionEnd styles are applied after new animation starts and accidentally overrides the latest motion values.

When new animation starts while animating, MotionValue stops the ongoing animation and immediately starts the new one. However as transitionEnd waits for all animations to be resolved, some transitionEnd styles overrides new values.

It seems to be introduced in #2025. Before it lands, calling stop() never resolves animation Promises, which are used to apply the transitionEnd styles at the end, so there was no race condition..

Closes: chakra-ui/chakra-ui#7775

[mattgperry]

Can you add a test for this behaviour? It also looks like this would stop all of transitionEnd values being applied if even any one of the values is animating.

[chuganzy]

@mattgperry Sure, added a test and fixed the logic to check if animations completed vs canceled here

[jonahallibone]

Any update on this PR? I believe I am experiencing the issue this PR would fix

[stefanweck]

This fix would resolve #2317. Any update on merging this fix?

[jonahallibone]

Bumping this, any update Framer team?

[AndreiRailean]

Hey @mattgperry what do you think needs to happen to get this PR merged?
I assume at a minimum it needs to be brought up-to-date with master. Is there anything else?

[TranslucentDragon]

Would like to bump this as well. I came here from the earlier linked accordion chakra UI issue. What else needs to happen to get this merged? @mattgperry

[jonahallibone]

It seems like this isn't a problem in the newest versions of framer motion. I haven't noticed it in in the most recent couple versions. Can anyone else confirm?