Image screen cover mode#201
Open
JorxPi wants to merge 3 commits into
Open
Conversation
AndreuMiroSabate
approved these changes
Jul 10, 2026
AndreuMiroSabate
left a comment
Contributor
There was a problem hiding this comment.
Good changes. This fix was needed, and now the UI resize is getting close to a completely good functionality. Keep the good work.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR improves UI resizing behaviour across different viewport and window sizes.
The UI was not resizing consistently across editor/game viewports and different window sizes. Some screen space UI elements were being calculated using the wrong viewport dimensions, and swapchain render surfaces was returning incorrect sizes which were depending on the last attached texture. This caused full screen UI backgrounds and menu layouts to appear cropped, offset, or scaled incorrectly.
The changes done are the following:
After all the improvement, here I put a video with the final result of the Main_Menu resizing correctly. It still needs one last change, which is detaching the Credits and Controlls text from the background, this way the background can use the Cover Mode and the text can use normal stretch.
https://youtu.be/rFnAnqz1dTY
I also attach you the new Main_Menu scene if you would like to test yourselves. Once this Pr is accepted I will move the scene to the BoundByDeath project.
Main_Menu.zip