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

Qt: Use better method of getting default monospace font #9085

Merged
merged 1 commit into from Sep 15, 2020

Conversation

Techjar
Copy link
Contributor

@Techjar Techjar commented Sep 15, 2020

This should give us a nicer font on Windows, while also not severely impacting the existing behavior on Linux.

At least on my Linux system, it actually results in using the monospace font I have configured (which is "Hack").

Before ("Monospace")

After ("Hack")

@JosJuice
Copy link
Member

We're using the same approach in the log widget. I assume it would make sense to change that as well?

f = QFont(QStringLiteral("Monospace"));
f.setStyleHint(QFont::TypeWriter);

@Techjar
Copy link
Contributor Author

Techjar commented Sep 15, 2020

Ah, I wasn't aware it was separate.

This should give us a nicer font on Windows, while also not severely
impacting the existing behavior on Linux.
@JMC47
Copy link
Contributor

JMC47 commented Sep 15, 2020

Before:

image

After:

image

And the after runs ~10X smoother.

@Techjar Techjar changed the title Qt/Settings: Use better method of getting default monospace font Qt: Use better method of getting default monospace font Sep 15, 2020
@lioncash lioncash merged commit 9e3c083 into dolphin-emu:master Sep 15, 2020
10 checks passed
@pizuz
Copy link

pizuz commented Sep 15, 2020

This fixes the startup delay observed in macOS as well.

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