fix(fonts): the app uses Inter again (#320) #370

Merged
jared merged 1 commits from fix/inter-font-name into lotus 2026-10-17 17:51:27 -04:00
Owner

Fixes #320.

Since the switch to @fontsource-variable/inter 5 in May, the package registers the font family 'Inter Variable', but the app still asked for the 4.x name 'InterVariable'.

  • Every user got their platform's default sans-serif instead: Arial on Windows, Helvetica on macOS, DejaVu or Liberation on Linux.
  • Settings → Appearance → font "Inter" did nothing.

Change: --font-secondary (index.css) and the App font map ask for 'Inter Variable' first, keeping 'InterVariable' as a fallback for a locally installed old version.

Verified headless:

  • Before: 0 Inter faces loaded, and document.fonts.check('16px "Inter Variable"') was false.
  • After: the face loads and the check is true.

This is a visible change for everyone, back to the font the design was tuned for. Screenshots below.

Before (system sans-serif)

Before (system sans-serif)

After (Inter)

After (Inter)

🤖 Generated with Claude Code

https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5

Fixes #320. Since the switch to `@fontsource-variable/inter` 5 in May, the package registers the font family **`'Inter Variable'`**, but the app still asked for the 4.x name `'InterVariable'`. - Every user got their platform's default sans-serif instead: Arial on Windows, Helvetica on macOS, DejaVu or Liberation on Linux. - Settings → Appearance → font "Inter" did nothing. **Change:** `--font-secondary` (`index.css`) and the App font map ask for `'Inter Variable'` first, keeping `'InterVariable'` as a fallback for a locally installed old version. **Verified** headless: - **Before:** 0 Inter faces loaded, and `document.fonts.check('16px "Inter Variable"')` was false. - **After:** the face loads and the check is true. This is a visible change for everyone, back to the font the design was tuned for. Screenshots below. **Before (system sans-serif)** ![Before (system sans-serif)](https://code.lotusguild.org/attachments/fd7047f7-8e64-42b4-9b7c-d514a58dae8a) **After (Inter)** ![After (Inter)](https://code.lotusguild.org/attachments/15facc40-0b5f-4eb1-9056-aa1f004fcfd1) 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
jared added 1 commit 2026-10-16 22:01:09 -04:00
fix(fonts): the app uses Inter again (#320)
CI / Build & Quality Checks (pull_request) Successful in 3m49s
CI / Trigger Desktop Build (pull_request) Skipped
CI / Docker image build & smoke test (pull_request) Skipped
CI / Secret scan (gitleaks) (pull_request) Successful in 13s
CI / Playwright smoke (e2e) (pull_request) Successful in 14m34s
37372b215a
Since the switch to @fontsource-variable/inter 5 (May), the package
registers the family 'Inter Variable', but the app still asked for the
4.x name 'InterVariable', so every user got their platform's default
sans-serif (Arial / Helvetica / DejaVu) and Settings → Appearance →
font "Inter" did nothing.

--font-secondary (index.css) and the App font map now ask for
'Inter Variable' first, keeping 'InterVariable' as a fallback for a
locally installed old version.

Verified headless: before, 0 Inter faces loaded and
document.fonts.check('16px "Inter Variable"') false; now the face loads
and the check is true.

Fixes #320

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_017PCb59uY3UKqDqt7B6hzd5
jared merged commit a3ef2b4098 into lotus 2026-10-17 17:51:27 -04:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: LotusGuild/cinny#370