Skip to content

Conversation

@johnzhou721
Copy link
Contributor

[Test URL: https://beewareorg--10.org.readthedocs.build/zh-cn/10/]

The Chinese rendering for headings right now has very thin characters as compared to the english characters in Cutive:

Screenshot 2026-01-28 at 19 19 23

This is due to the fact that the default MkDocs theme wants H2 to be 300 weight, but Cutive does not have that thin of a weight, so it falls back onto the regular font weight, which is very thick for a "Regular" or 300 font. Since not only Chinese but other non-Cutive fonts is likely to have this problem, I've bumped the weight for H1 and H2 to 400.

In addition, the serifs looks out of place in Chinese because the style (Songti / Mingti; I don't know which one) doesn't match the slab-serif style of Cutive. Sans-Serif matches better, so I've added a variable for Chinese to use sans-serif in headings as well.

@freakboy3742 @kattni I'm interested in hearing thoughts on this change. For reference, sans-serif Chinese with 400 weight:

Screenshot 2026-01-28 at 19 19 04

Serif Chinese with 400 weight:

Screenshot 2026-01-28 at 19 24 55

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

@kattni
Copy link
Collaborator

kattni commented Jan 29, 2026

@johnzhou721 PR #150 is part of the website migration. As it affects all the documentation, and there is a change in the header, we are waiting to take it live until the new website is live. In that PR, I have gone to great lengths to disable the dynamic font sizing in the MkDocs Material theme. This means that every header font size is defined explicitly in rem. I understand where you're going with this, but with everything in flux, this is not the time to make this change. I'm converting this to a draft so it doesn't get merged by someone else who doesn't have the context. When we take things live, please test your issues against that setup, and we can discuss it at that point if needed.

@kattni kattni marked this pull request as draft January 29, 2026 01:55
@johnzhou721
Copy link
Contributor Author

@kattni Sorry, I apologize for not seeing the conflict with #150. And also I meant to mark this as draft, as I labeled it PROPOSAL, but I somehow clicked the wrong button. This also affects the tutorial, and I apologize for neglecting to concern that as part of the website migration there was also a Docs Tools PR.

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