X Tutup
Skip to content

Fix cinematic preview causing the editor redraw continuously and aspect ratio not updating in camera preview#109469

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Open-Industry-Project:fix-previews
Aug 12, 2025
Merged

Fix cinematic preview causing the editor redraw continuously and aspect ratio not updating in camera preview#109469
Repiteo merged 1 commit intogodotengine:masterfrom
Open-Industry-Project:fix-previews

Conversation

@ryevdokimov
Copy link
Contributor

Fixes: #101499

Moves the logic out of notification.

The aspect ratio yellow rectangle updating worked with cinematic preview because of the bug (but didn't work with just regular preview camera), this PR also makes it work with both after the redraw fix by making it driven by the camera properties changing.

2025-08-09.21-20-19.mp4

@AThousandShips AThousandShips added bug topic:editor topic:3d performance cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release labels Aug 11, 2025
@AThousandShips AThousandShips added this to the 4.5 milestone Aug 11, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Tested locally (together with #109490), it works as expected. Code looks good to me.

@Repiteo Repiteo merged commit 8aa37ca into godotengine:master Aug 12, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Aug 12, 2025

Thanks!

@ryevdokimov ryevdokimov deleted the fix-previews branch August 12, 2025 23:12
atlasapplications pushed a commit to atlasapplications/godot that referenced this pull request Aug 13, 2025
Fix cinematic preview causing the editor redraw continuously and aspect ratio not updating in camera preview
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release performance topic:editor topic:3d

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cinematic Preview makes the editor redraw continuosly

4 participants

X Tutup