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

Allow overriding the loop flag on player_api.set_animation #3097

Merged
merged 2 commits into from
Jan 28, 2024

Conversation

Panquesito7
Copy link
Contributor

Things added/changed:

Copy link
Contributor

@appgurueu appgurueu left a comment

Choose a reason for hiding this comment

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

Added a commit to implement loop analogeously to speed. In particular, it needs to be remembered, otherwise you wouldn't be able to change loop when setting the same animation.

I haven't this tested yet however, make sure it still works for you. I'll test this later.

@Panquesito7
Copy link
Contributor Author

Added a commit to implement loop analogeously to speed. In particular, it needs to be remembered, otherwise you wouldn't be able to change loop when setting the same animation.

I haven't this tested yet however, make sure it still works for you. I'll test this later.

Thanks! Tested it, works perfectly.

@sfan5 sfan5 merged commit 222afb0 into minetest:master Jan 28, 2024
1 check passed
@Panquesito7 Panquesito7 deleted the animation_loop_flag branch January 29, 2024 04:56
MoNTE48 pushed a commit to MoNTE48/minetest_game that referenced this pull request Jun 2, 2024
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.

Allow overriding the loop flag on player_api.set_animation
3 participants