Skip to content

fix: load docs font from subpath - #139

Merged
Tom Tang (qwerzl) merged 1 commit into
mainfrom
agent/fix-docs-font-path
Aug 25, 2026
Merged

fix: load docs font from subpath#139
Tom Tang (qwerzl) merged 1 commit into
mainfrom
agent/fix-docs-font-path

Conversation

@qwerzl

@qwerzl Tom Tang (qwerzl) commented Aug 25, 2026

Copy link
Copy Markdown
Member

Summary

  • Point the heading and body font sources at /docs/fonts/PolySansArabicVF.woff2.
  • Avoid requests to the root /fonts route, which returns 404 on photon.codes.

Validation

  • pnpm lint
  • pnpm typecheck:docs
  • npx -y mint@latest broken-links
  • npx -y mint@latest validate
  • Confirmed the /docs/fonts asset returns 200 with font/woff2.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Bug Fixes
    • Corrected documentation font paths so headings and body text load properly.

Copilot AI lite review requested due to automatic review settings August 25, 2026 07:06

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d1cbabf8-2c5a-42c9-88a0-00dd73995af8

📥 Commits

Reviewing files that changed from the base of the PR and between 7a98871 and b9dd6d9.

📒 Files selected for processing (1)
  • docs.base.json

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

The documentation configuration now loads the heading and body fonts from /docs/fonts/ instead of /fonts/.

Changes

Font path update

Layer / File(s) Summary
Update documentation font paths
docs.base.json
The heading and body font sources now use /docs/fonts/PolySansArabicVF.woff2.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: lcandy2

Poem

A rabbit checks the font path bright
Two styles now hop to /docs/fonts/ right
The family stays, the format too
PolySansArabic shines anew
Documentation blooms in view

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch agent/fix-docs-font-path

Comment @coderabbitai help to get the list of available commands.

@qwerzl
Tom Tang (qwerzl) merged commit 9950537 into main Aug 25, 2026
5 of 6 checks passed
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