Skip to content

Use Inter for app typography#899

Merged
klopez4212 merged 4 commits into
mainfrom
kennylopez-inter-font-test
Jun 17, 2026
Merged

Use Inter for app typography#899
klopez4212 merged 4 commits into
mainfrom
kennylopez-inter-font-test

Conversation

@klopez4212

Copy link
Copy Markdown
Contributor

Summary

  • Switch desktop and web sans typography to bundled Inter Variable
  • Update Tailwind font-sans stacks to match the global app font
  • Fix the staging mesh preset lookup after the ACP runtime rename

Checks

  • pnpm --filter ./desktop build
  • pnpm --filter ./web build
  • pnpm --filter ./desktop check
  • pnpm --filter ./web check
  • just staging

@klopez4212 klopez4212 force-pushed the kennylopez-inter-font-test branch from aee88ee to 7d128cf Compare June 17, 2026 12:03
@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 Inter font screenshots from the rebased branch.

Home inbox

Inbox list plus selected thread detail, useful for dense scanning text.
01-home-inbox

Channel timeline

#general timeline, sidebar labels, message text, and composer.
02-channel-general

Direct message

DM header, sidebar labels, conversation area, and composer.
03-dm-alice-tyler

klopez4212 added a commit that referenced this pull request Jun 17, 2026
@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 Line-height/alignment check done after the Inter swap.

Measured the old font stack against Inter in the channel sidebar, DM row, message mention chips, channel-link chips, inline code chips, and composer mention chips. Sidebar icon/text centers stayed aligned in both stacks. The agent/app mention icon was the one measurable drift: it sat about 0.85px high because the pseudo-icon used a fixed top offset.

Pushed a small fix in d8e2456 to center that pseudo-icon with top: 50% + translateY(-50%). Re-measurement puts the agent icon offset at effectively 0px in both timeline and composer chips. Direct Biome lint on the touched CSS file passed.

@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 Follow-up line-height pass for wrapped Inter text.

I measured a deliberately wrapped timeline message and wrapped composer draft. Before this change, both used 22.75px line-height with Inter glyph bounds around 17px, which made the gaps between wrapped lines feel roomier than the old stack.

Pushed 4b7005c to tighten default message markdown and rich-text composer rhythm to the existing text-sm / leading-5 rhythm (20px). Paragraphs now inherit the markdown container line-height, so compact callers like inbox/feed previews can keep their own tighter rhythm instead of being forced back to relaxed. Letter spacing remains normal; I did not add tracking changes.

@klopez4212

Copy link
Copy Markdown
Contributor Author

🤖 Fresh Inter screenshots after the line-height/alignment updates.

Home inbox

Inbox list and selected thread detail with the final Inter rhythm.
01-home-inbox

Channel timeline

#general with sidebar, channel intro, message text, chips, and composer.
02-channel-general

Direct message

DM header, sidebar label, empty-state copy, and composer.
03-dm-alice-tyler

Wrapped text + composer

Seeded wrapped message and wrapped draft showing 14px / 20px text rhythm and normal letter spacing.
04-wrapped-text-composer

klopez4212 added a commit that referenced this pull request Jun 17, 2026
@klopez4212 klopez4212 marked this pull request as ready for review June 17, 2026 14:18
@klopez4212 klopez4212 requested a review from a team June 17, 2026 14:19

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4b7005ca62

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread desktop/src/shared/styles/globals.css Outdated
@klopez4212 klopez4212 force-pushed the kennylopez-inter-font-test branch from 8b14029 to 1d85dee Compare June 17, 2026 15:28
@klopez4212 klopez4212 merged commit 7b0500c into main Jun 17, 2026
28 of 29 checks passed
@klopez4212 klopez4212 deleted the kennylopez-inter-font-test branch June 17, 2026 17:56
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