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

naively scale dialogs by ui scale #351

Closed
wants to merge 1 commit into from

Conversation

NQNStudios
Copy link
Collaborator

This is a naive solution to #350, dialog scaling. It just scales up the dialog after drawing text (so it looks blurry), and divides mouse coordinates by scale so you can still click on everything.

It could definitely have bugs in edge cases of subclasses of Dialog, or if any dialogs don't use the Dialog class, some windows won't be fixed.

@CelticMinstrel
Copy link
Member

Looks fine to me, though you may want to consider putting get_float_pref("UIScale", 1.0) into a private function in cDialog, since it's used at least 3 times.

Doesn't the main UI scaling also scale up after text? It should end up being pixellated rather than blurry, though.

@CelticMinstrel
Copy link
Member

It could definitely have bugs in edge cases of subclasses of Dialog, or if any dialogs don't use the Dialog class, some windows won't be fixed.

I don't think there are any that don't use the Dialog class, and as for subclasses, I'm pretty sure there aren't any.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants