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

PPGe: Scale down by worst of window/internal res #13962

Merged
merged 1 commit into from
Jan 23, 2021

Conversation

unknownbrackets
Copy link
Collaborator

@unknownbrackets unknownbrackets commented Jan 23, 2021

If you have a really low render resolution but a high window resolution, we were allowing more scale down than intended. Though, it still looked more readable for me than #13958.

This just takes the lowest of the two when considering how much to shrink text. This only happens when there's a lot of text, so most games will be unchanged.

-[Unknown]

@hrydgard hrydgard added the HLE/Kernel Kernel, memory manager, other HLE issues label Jan 23, 2021
@hrydgard hrydgard added this to the v1.11.0 milestone Jan 23, 2021
@hrydgard
Copy link
Owner

Hm. I'm of two minds whether it's right to involve the window resolution at all in this calculation. But should be OK.

@hrydgard hrydgard merged commit b856b5c into hrydgard:master Jan 23, 2021
@unknownbrackets unknownbrackets deleted the ppge-shrink branch January 23, 2021 18:15
@unknownbrackets
Copy link
Collaborator Author

Well, ignoring it gives you this:

Trails in the Sky with extra lines in savedata description

Instead of:

Trails in the Sky with fewer lines in savedata description

At 2x or higher it's small and probably hard to read, but at least it can be read:

Trails in the Sky with extra lines at a higher resolution

We could just truncate everyone and refuse to make it smaller ever, but it's nice to be able to get to the extra detail. To be fair, probably a lot of people ignore this text anyway, and only a small percent of games seem to use these longer descriptions.

-[Unknown]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HLE/Kernel Kernel, memory manager, other HLE issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants