Fix viewport attach position when forced integer stretch scale of 1 makes it bigger than window #91787
+8
−5
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.
Fixes #89578.
Supersedes #90257.
Haven't dived deep into the code (looked only into
Window::_update_viewport_size
) so please test thoroughly. Firstly I've tried just adding a proper offset to themargin
, it resulted in the opposite issue compared to #89578 (comment):Then I've noticed
margin
is also applied towindow_transform
, hence tried a separate additional offset just forattach_to_screen_rect
. And that's this PR, seems to work. 🙃