8000 fonts: Use `Helvetica` as the `system-ui` font on macOS by dpogue · Pull Request #39001 · servo/servo · GitHub
[go: up one dir, main page]

Skip to content

Conversation

dpogue
Copy link
Contributor
@dpogue dpogue commented Aug 28, 2025

Helvetica isn't quite right but it's a better default choice than a monospaced font like Menlo (it should be some variant of Apple's San Francisco font, but that isn't easily exposed)

Testing: Untested, but matches the font family used for sans-serif so it should be safe

Helvetica isn't quite right but it's a better default choice than a monospaced font like Menlo (it should be some variant of Apple's San Francisco font, but that isn't easily exposed)

Signed-off-by: Darryl Pogue <darryl@dpogue.ca>
@yezhizhen yezhizhen changed the title fonts: Better system-ui font choice on macOS font(macOS): Use Helvetica instead of Menlo as system-ui Aug 29, 2025
@yezhizhen yezhizhen changed the title font(macOS): Use Helvetica instead of Menlo as system-ui font (macOS): Use Helvetica instead of Menlo as system-ui Aug 29, 2025
@yezhizhen yezhizhen changed the title font (macOS): Use Helvetica instead of Menlo as system-ui font (macOS): Use Helvetica instead of Menlo as default family for system-ui Aug 29, 2025
@yezhizhen yezhizhen requested a review from nicoburns August 29, 2025 09:45
@mrobinson mrobinson changed the title font (macOS): Use Helvetica instead of Menlo as default family for system-ui fonts: Use Helvetica as the system-ui font on macOS Aug 29, 2025
Copy link
Member
@mrobinson mrobinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Eventually we should create the font in the same way that Gecko does, but this seems like a definite improvement for now.

@mrobinson mrobinson enabled auto-merge August 29, 2025 10:47
@mrobinson mrobinson added this pull request to the merge queue Aug 29, 2025
Merged via the queue into servo:main with commit 89e1357 Aug 29, 2025
26 checks passed
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
0