Skip to content

feat: Use system-ui fonts#4001

Draft
tvdeyen wants to merge 19 commits into
mainfrom
use-system-ui-font
Draft

feat: Use system-ui fonts#4001
tvdeyen wants to merge 19 commits into
mainfrom
use-system-ui-font

Conversation

@tvdeyen

@tvdeyen tvdeyen commented Jun 22, 2026

Copy link
Copy Markdown
Member

What is this pull request for?

The fastest fonts available.

No downloading, no layout shifts, no flashes — just instant renders.

Cuts the admin.css file into half (364KB -> 182KB)!
And the welcome.css from 186KB to 5KB (!!)

180KB just fonts. On every request. We do not need that.

https://modernfontstacks.com/

Notable changes (remove if none)

Raises the font size from 12px to 13px and lowered the font weight for bold fonts to 600

Screenshots

Remove if no visual changes have been made.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen added this to the 8.3 milestone Jun 22, 2026
@tvdeyen tvdeyen added the enhancement New feature or enhancement label Jun 22, 2026
@codecov

codecov Bot commented Jun 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.12%. Comparing base (4b64e17) to head (cebb697).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4001   +/-   ##
=======================================
  Coverage   98.12%   98.12%           
=======================================
  Files         346      346           
  Lines        8974     8974           
=======================================
  Hits         8806     8806           
  Misses        168      168           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tvdeyen tvdeyen removed this from the 8.3 milestone Jun 23, 2026
@tvdeyen tvdeyen self-assigned this Jun 23, 2026
tvdeyen added 19 commits June 23, 2026 21:11
The fastest fonts available.

No downloading, no layout shifts, no flashes — just instant renders.

Cuts the admin.css file into half (364KB -> 182KB)!
And the welcome.css from 186KB to 5KB (!!)

180KB just fonts. On every request. We do not need that.

https://modernfontstacks.com
A bit more now that we use system fonts, which can be
a bit small.
Now that we use a system-font the font-weight is
too heavy.
Instead of individual places, we just enable it globally.
The labels are baseline, so the value need as well and the icon needs adjusment
Better legibility
Instead of the button, which hasd a fixed height the small
button should use the height calculated by padding and
font-size.
This is useless for fonts under 20px and does not
really make any visual difference
@tvdeyen
tvdeyen force-pushed the use-system-ui-font branch from 0a8cd64 to 9cd3289 Compare June 24, 2026 09:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants