fix(theme): remove some explicit fonts

vuejs / vitepress

Vite & Vue powered static site generator.

https://vitepress.dev

MIT License

13.22k stars 2.13k forks source link

fix(theme): remove some explicit fonts #4286

Closed Jungzl closed 1 month ago

Jungzl commented 1 month ago

Description

These fonts before sans-serif have poor rendering on Windows, remove them so that users who don't have PingFang SC & Noto Sans SC' installed can still set their preferred fonts.

[!TIP] The author of this PR can publish a preview release by commenting /publish below.

Jungzl commented 1 month ago

/publish

pkg-pr-new[bot] commented 1 month ago

pnpm add https://pkg.pr.new/vitepress@4286

commit: 5cc5ead

brc-dd commented 1 month ago

Need someone to confirm this on Chrome on Windows 10 or 11 (who hasn't customized fonts in Chrome/Win settings) - what does this show:

Open https://vitepress.dev/zh/
Open devtools
Select <body>
Uncheck "font-family" in "Styles" tab:
Select any Chinese text
Go to "Computed" tab in devtools
Scroll down to "Rendered Fonts":

Jungzl commented 1 month ago

online version:

local version:

UA: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/132.0.0.0 Safari/537.36
OS: Windows 11