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.

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

Fixes Reverse Animation Starting on First Frame #31943

Merged

Conversation

codecustard
Copy link
Contributor

When playing an animation in reverse, the animation initially starts on frame 0. If it loops, it'll play normally by going to the last frame of the animation, but if it does not... it prematurely stops, since it is already on the last frame (for reversed animation) by starting on frame 0.

Closes: #31903

@akien-mga
Copy link
Member

This could be problematic if you want to play the same animation forward and then backward, changing at an intermediate position.

@akien-mga akien-mga added this to the 3.2 milestone Sep 24, 2019
When playing an animation in reverse, the animation initially starts on frame 0. If it loops, it'll play normally by going to the last frame of the animation, but if it does not... it prematurely stops, since it is already on the last frame (for reversed animation) by starting on frame 0.
@codecustard codecustard force-pushed the Fixes_invalid_frames_on_reverse_anim branch from dc5bc31 to 94a00cd Compare October 16, 2019 11:26
@codecustard
Copy link
Contributor Author

codecustard commented Oct 16, 2019

This could be problematic if you want to play the same animation forward and then backward, changing at an intermediate position.

Updated PR to check initial frame.

@akien-mga akien-mga merged commit 2416fbb into godotengine:master Oct 27, 2019
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setting backwards boolean to true in AnimatedSprite.play() emits animation_finished prematurely
3 participants