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

Handle portrait mode monitors in the automatic editor scale detection #48597

Merged
merged 1 commit into from
Jun 15, 2021

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented May 9, 2021

Using the smallest dimension of the width and height makes it possible to support both landscape and portrait monitors.

This closes #48585.

@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:editor usability labels May 9, 2021
@Calinou Calinou added this to the 4.0 milestone May 9, 2021
Using the smallest dimension of the width and height makes it possible
to support both landscape and portrait monitors.
@Calinou Calinou force-pushed the editor-scale-auto-portrait branch from b643b56 to 728fa3f Compare May 9, 2021 21:35
@lewiji
Copy link
Contributor

lewiji commented May 10, 2021

Have just built and tested with this change and yep, Godot uses the same UI scale for both screens now 😊 I noted that both were detected to use 150%, unlike 3.3 which detects 100% for my primary screen, not sure if that's just a change in 4. Also of note - the window fit correctly on the secondary screen, even at 150% scaling.

@Calinou
Copy link
Member Author

Calinou commented May 10, 2021

I noted that both were detected to use 150%, unlike 3.3 which detects 100% for my primary screen, not sure if that's just a change in 4.

Maybe this is because the primary/startup monitor ID is detected differently.

@lewiji
Copy link
Contributor

lewiji commented May 10, 2021

Yes, that's possible, and also it looked fine and usable to me and not like it was too big.

@Calinou Calinou marked this pull request as ready for review May 10, 2021 12:36
@akien-mga akien-mga merged commit 3f01b38 into godotengine:master Jun 15, 2021
@akien-mga
Copy link
Member

Thanks!

// hiDPI display.
scale = 2.0;
} else if (DisplayServer::get_singleton()->screen_get_size(screen).y >= 1700) {
} else if (smallest_dimension <= 1700) {
Copy link
Member

Choose a reason for hiding this comment

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

That change is bogus, no? Should be >= 1700.

Copy link
Member

Choose a reason for hiding this comment

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

Should be fixed in #49654.

@akien-mga
Copy link
Member

Cherry-picked for 3.4. (Together with #49654.)

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Jun 17, 2021
@Calinou Calinou deleted the editor-scale-auto-portrait branch August 3, 2021 15:59
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.

"Auto" display scale on portrait vs landscape monitors is too large
3 participants