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

fix: font display is irregular for WeaselDeployer. #1454

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

alpzmj9
Copy link

@alpzmj9 alpzmj9 commented Dec 18, 2024

The current font is set to “MS Shell Dlg”, which is mapped to “SimSun” on Chinese operating systems instead of the system default font, which throws to very terrible display results.

I just changed it to "Segoe UI" and the problem was solved.

Q: Why not set it to "Microsoft YaHei" or "Microsoft JhengHei" immediately?

A: Since many Chinese users have both fonts on their computers, doing so will most likely result in them being forced to be mapped to the wrong font.

Q: Why Segoe UI?

A: Because it's Windows's default font for the English version, than can be a fallback to correct fonts for other languages referenced by system fontlink.

The current font is set to “MS Shell Dlg”, which is mapped to “SimSun” on Chinese operating systems instead of the system default font, which throws to very terrible display results.

I just changed it to "Segoe UI" and the problem was solved.

Q: Why not set it to "Microsoft YaHei" or "Microsoft JhengHei" immediately?

A: Since many Chinese users have both fonts on their computers, doing so will most likely result in them being forced to be mapped to the wrong font.

Q: Why Segoe UI?

A: Because it's Windows's default font for the English version, than can be a fallback to correct fonts for other languages referenced by system fontlink.
@fxliang
Copy link
Contributor

fxliang commented Dec 18, 2024

it's not SimSun, but logic font MS Shell Dlg, this is the recommended approach from Microsoft.

in my windows 10 laptop in Chs, it's mapped to MS Sans Serif

you can check HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\FontSubstitutes for the font in your system.

for detail information, check this https://learn.microsoft.com/zh-cn/windows/win32/intl/using-ms-shell-dlg-and-ms-shell-dlg-2

@alpzmj9
Copy link
Author

alpzmj9 commented Dec 18, 2024

it's not SimSun, but logic font MS Shell Dlg, this is the recommended approach from Microsoft.

in my windows 10 laptop in Chs, it's mapped to MS Sans Serif

you can check HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\FontSubstitutes for the font in your system.

for detail information, check this https://learn.microsoft.com/zh-cn/windows/win32/intl/using-ms-shell-dlg-and-ms-shell-dlg-2

I think you're right, but it does end up being mapped to "SimSun" via Fontlink.

I have checked the Microsoft documentation and this may be a bug from Microsoft, but it is definitely not normal and has been there for a long time.

you can check "Microsoft Sans Serif" registry item in "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows NT\CurrentVersion\FontLink\SystemLink"

This is a screenshot of my registry item, also Win10/Chs, OS should be the latest version, and I have not modified it.
image

@alpzmj9
Copy link
Author

alpzmj9 commented Dec 18, 2024

屏幕截图 2024-12-18 232747 屏幕截图 2024-12-18 232839 Attach a before and after comparison (the monitor is 2160P/150 scaling).

If the screen resolution is below 1080P and there is no scaling, "SimSun" will be dotted, maybe resulting in a relatively good display effect.

But as soon as the resolution is greater than 1080 and the scaling is non-integer ratio, it starts to get really bad.

@lotem
Copy link
Member

lotem commented Dec 19, 2024

有趣的研究。右邊看上去清楚一些。

@lotem
Copy link
Member

lotem commented Dec 19, 2024

我記得Windows以前用那種點陣宋體,默認字號還挺清楚的。微軟瞎改?

@fxliang
Copy link
Contributor

fxliang commented Dec 19, 2024

我記得Windows以前用那種點陣宋體,默認字號還挺清楚的。微軟瞎改?

好像是因为DPI修改之后的问题,150%比例似乎是会有点虚

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.

3 participants