Repository navigation
docs: professional typography and UI polish for documentation pages - #171
Merged
Merged
Conversation
Set titles, headings and body in Inter on every page with a sidebar (the condensed display face stays on the homepage), with a calmer scale, body copy one step below the heading color, a 66ch measure and consistent heading spacing. Use IBM Plex Mono for all code: --sl-font-mono named JetBrains Mono, which the site never loaded. Keep heading anchors and code copy buttons quiet until hover or focus, give sidebar badges one style, set table headers in small caps, wrap long "On this page" entries (the panel was sized from the 18rem left sidebar), and give phone prose, headings, code and callouts one gutter. The typography lives in docs-theme.css, where the .not-content guard test checks it; the superseded heading rules in docs-reading.css and docs-theme.css are removed. On docs-only changes, CI now also runs the tests that locate the docs through path segments (join(root, 'docs', 'src')). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--sl-font-mononamed JetBrains Mono, which was never loaded.join(root, 'docs', 'src')), such as the.not-contentguard for theme rules.Typography lives in
docs-theme.csssotest/docs-theme-not-content.test.ts(unchanged) keeps checking it; superseded heading rules are removed. Checked with screenshots in light/dark, desktop/mobile, on guides, architecture, blog and home (unchanged). No version bump, no source or test changes.🤖 Generated with Claude Code