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 Camera2D frame delay (port from 3.x) #84465

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

KeyboardDanni
Copy link
Contributor

@KeyboardDanni KeyboardDanni commented Nov 5, 2023

This is a port of #46697 to the 4.x branch.

Complements #84380 by further improving the 2D pixel perfect experience.

Test project (to be used in conjunction with PR 84380): PixelPerfectTestIdle.zip

Fixes #74203 and fixes #77813.

@AThousandShips AThousandShips added this to the 4.x milestone Nov 5, 2023
@KeyboardDanni KeyboardDanni marked this pull request as ready for review November 5, 2023 19:33
@KeyboardDanni KeyboardDanni requested a review from a team as a code owner November 5, 2023 19:33
@KeyboardDanni KeyboardDanni changed the title Deferred Camera2D processing to fix one-frame lag in some scenarios Fix Camera2D frame delay (port from 3.x) Nov 5, 2023
Copy link
Member

@adamscott adamscott left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@akien-mga akien-mga added the bug label Jan 12, 2024
@akien-mga akien-mga merged commit ebc30b9 into godotengine:master Jan 15, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks! And congrats for your first merged Godot contribution 🎉

@KeyboardDanni
Copy link
Contributor Author

Technically not my first contribution, but thank you!

@akien-mga
Copy link
Member

Oops, my bad indeed. Well, welcome back! :D

@kleonc
Copy link
Member

kleonc commented Mar 5, 2024

Another issues fixed by this: #73298, #89174.

Might consider cherry-picking it to 4.2.2, not sure if it's "safe enough" though. Up to you @akien-mga. 😉

@akien-mga akien-mga added the cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release label Mar 5, 2024
@akien-mga
Copy link
Member

I guess it's worth a try? I'll cherry-pick it for 4.2.2 RC 2.

@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 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants