Skip to content

Stop the Mac's terminal text from looking heavy - #39

Closed
corgan-1024 wants to merge 1 commit into
owngoal-dev:mainfrom
corgan-1024:mac-font-thicken-off
Closed

corgan-1024 wants to merge 1 commit into
owngoal-dev:mainfrom
corgan-1024:mac-font-thicken-off

Conversation

@corgan-1024

Copy link
Copy Markdown

On Mac, Latin text and PingFang Chinese draw visibly heavier than in Ghostty or other terminals using the same JetBrains Mono; the Chinese in particular reads like Linux rendering.

  • libghostty-spm's TerminalConfiguration.default calls withFontThicken(true), so every generated config starts with font-thicken = true. Ghostty's own default is false.
  • With it on, ghostty enables CoreText font smoothing for each glyph (setShouldSmoothFonts in font/face/coretext.zig), which is macOS's stroke dilation.
  • The app's overlay never said otherwise, so the Mac inherited it.

This sets font-thicken = false in the Catalyst branch of GhosttyAppConfiguration.terminal, after the library's base so it wins. iOS is untouched: I did not compare the smoothing there, so its binary is unchanged.

Checked: adding font-thicken = false under Settings ▸ Ghostty Configuration on a Mac and opening a new tab made the text render normally; the Mac Catalyst build compiles with this change.

Not checked: iPhone and iPad rendering, and a side-by-side of the same Chinese text before and after.

If you would rather change the default in libghostty-spm so every consumer gets it, this overlay line can simply go.

On Mac Catalyst, Latin text and PingFang Chinese drew visibly heavier
than in Ghostty or other terminals using the same JetBrains Mono.

libghostty-spm's base configuration sets `font-thicken = true`, which makes
ghostty turn on CoreText font smoothing for every glyph; ghostty's own
default is false. The app's overlay never said otherwise, so the Mac
inherited it. Turn it off on Catalyst only; iOS is left as it was, since
the smoothing was not compared there.
@corgan-1024

Copy link
Copy Markdown
Author

Closing: this was opened from the wrong account. It will be reopened from the personal fork.

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