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

Fix animated tile random start time not scaled to animation duration #86210

Conversation

kleonc
Copy link
Member

@kleonc kleonc commented Dec 15, 2023

For TileMap's animated tiles random animation offset passed to the RenderingServer were not scaled to the animation duration, a normalized value (in 0.0..1.0 range) were passed instead, see godotengine/godot-docs#8595 (comment) for the issue in action.

cc @rakkarage

@dannflor
Copy link

🤞manifesting cherry pick for 4.2.2🤞

@kleonc kleonc added the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Dec 15, 2023
@Chaosus Chaosus added this to the 4.3 milestone Dec 28, 2023
@kleonc kleonc force-pushed the tilemap-animated-tile-unnormalize-random-start-time branch from 7d86964 to 7dfd0c4 Compare February 11, 2024 12:16
@kleonc
Copy link
Member Author

kleonc commented Feb 11, 2024

Rebased.

@akien-mga akien-mga merged commit 860dbdf into godotengine:master Feb 12, 2024
16 checks passed
@akien-mga
Copy link
Member

Thanks!

@kleonc kleonc deleted the tilemap-animated-tile-unnormalize-random-start-time branch February 12, 2024 13:26
@akien-mga
Copy link
Member

Cherry-picked for 4.2.2.

@akien-mga akien-mga removed the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Mar 11, 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.

5 participants