Skip to content

Load the Inter font and improve text readability - #209

Open
bendichter wants to merge 8 commits into
mainfrom
typography
Open

bendichter wants to merge 8 commits into
mainfrom
typography

Conversation

@bendichter

Copy link
Copy Markdown
Contributor

Combines three PRs about body text. Each one is a separate merge commit, so they can be reviewed one at a time.

  • Load the Inter font and use it for body text (#202): every font-family rule names Inter, but its @import came after other rules and was ignored, so pages fell back to Helvetica, Arial or the system font. The font is now requested from the page head and set on body. Pages grow by up to 5% in height.
  • Cap running text at about 80 characters per line (#204): paragraphs and list items are limited to 40em. Those holding an image, video, embed, table or code block keep the full width.
  • Darken the body text of news posts (#203): from #737780 (4.49:1 on white, just under WCAG AA) to #374151 (10.3:1), the color other templates use.

The combined branch builds with Hugo 0.152.2 and produces the same 165 pages as main.

Supersedes #202, #203 and #204.

🤖 Generated with Claude Code

bendichter and others added 8 commits October 9, 2026 15:11
Every font-family rule in style.css names Inter, but the font never
loaded. Its `@import` sat in the middle of the stylesheet, and browsers
ignore an `@import` that comes after other rules, so headings, lists and
navigation fell back to Helvetica or Arial. Paragraphs written in
Markdown have no inter-* class and used Bootstrap's system font stack,
so a page could show two typefaces.

The font is now requested from the page head, with a preconnect, and
`body` is set to Inter so that all text uses it.

Inter is slightly wider than the fallback fonts. The header button would
wrap to two lines between 1200 and 1400 px, so it is marked nowrap.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Paragraphs have no width limit, so on a desktop window they stretch
across the whole container. On the About, Team, FAQ and tool pages that
is 130 to 170 characters per line, about twice what is comfortable to
read.

Paragraphs and list items are now capped at 40em, measured in their own
font size. Those that hold an image, video, embed, table, code block or
nested layout are left alone, as are headings, cards and grids. Centered
paragraphs get auto margins so that they stay centered. Nothing changes
at phone width, where lines are already short.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The body of a news post is set in #737780, which has a contrast ratio
of 4.49:1 on white, just under the 4.5:1 that WCAG AA asks for. News
posts now use #374151 (10.3:1), the color the other templates use for
body text.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@netlify

netlify Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for nwborg ready!

Name Link
🔨 Latest commit e5460bb
🔍 Latest deploy log https://app.netlify.com/projects/nwborg/deploys/6acbcae28f02520008971ee9
😎 Deploy Preview https://deploy-preview-209--nwborg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@bendichter

Copy link
Copy Markdown
Contributor Author

Before and after

"Before" is the production site and "after" is this PR's Netlify deploy preview, both at 1440 px wide.

FAQ, "Why HDF5": Inter replaces the fallback font and lines are capped at about 80 characters

Before After
before after

News post: darker body text, plus the font and line length changes

Before After
before after

About NWB: Inter in the navigation, heading and body

Before After
before after

🤖 Generated with Claude Code

@bendichter
bendichter enabled auto-merge October 13, 2026 23:07

This branch has not been deployed

No deployments
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.

1 participant