Skip to content

fix(website): add font-sans to body styles#4392

Merged
NicholasKissel merged 2 commits intomainfrom
NicholasKissel/fix-site-font
Mar 10, 2026
Merged

fix(website): add font-sans to body styles#4392
NicholasKissel merged 2 commits intomainfrom
NicholasKissel/fix-site-font

Conversation

@NicholasKissel
Copy link
Member

Summary

  • Adds font-sans to the body element's Tailwind classes in website/src/styles/main.css, ensuring the website body uses the configured sans-serif font family instead of the browser default.

Test plan

  • Verify the website renders with the correct sans-serif font
  • Check that no other font styles are broken by the change

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@railway-app railway-app bot temporarily deployed to rivet-frontend / rivet-pr-4392 March 10, 2026 00:27 Destroyed
@railway-app
Copy link

railway-app bot commented Mar 10, 2026

🚅 Deployed to the rivet-pr-4392 environment in rivet-frontend

Service Status Web Updated (UTC)
frontend-inspector 😴 Sleeping (View Logs) Web Mar 10, 2026 at 12:37 am
website 🕒 Building (View Logs) Web Mar 10, 2026 at 12:36 am
frontend-cloud 😴 Sleeping (View Logs) Web Mar 10, 2026 at 12:34 am
mcp-hub ✅ Success (View Logs) Web Mar 10, 2026 at 12:27 am
ladle ❌ Build Failed (View Logs) Web Mar 10, 2026 at 12:27 am

@claude
Copy link

claude bot commented Mar 10, 2026

PR Review: fix(website): add font-sans to body styles

This PR makes two focused changes:

  1. Adds font-sans to the body element Tailwind classes in main.css
  2. Refactors blog/index.astro to align its styling with the existing changelog/index.astro pattern

Positives

  • Correct fix: Adding font-sans to the body is the right approach for ensuring Tailwind's configured sans-serif font is applied globally instead of the browser default.
  • Unused imports removed: Dropping Badge and Button from @rivet-gg/components is good housekeeping since they are no longer used.
  • Style parity with changelog: The blog page now mirrors the visual style of changelog/index.astro. The selection-dark class is defined in main.css and used consistently across other marketing pages.
  • Clean commit messages: Conventional commit format is followed correctly.

Minor Observations

Article heading weight reduced: The article h3 was changed from font-bold text-lg to font-normal text-base. Both font size and weight were reduced simultaneously. Worth a visual check to confirm article titles remain prominent in the card grid.

Author name styling simplified: The font-semibold wrapper div around the author name was removed. Worth confirming this matches the intended design.

Verdict

Looks good overall. The core font-sans fix is correct, the blog-to-changelog style alignment is coherent, and the code is clean. The minor observations above are not blockers.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Mar 10, 2026

More templates

@rivetkit/cloudflare-workers

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/cloudflare-workers@4392

@rivetkit/framework-base

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/framework-base@4392

@rivetkit/next-js

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/next-js@4392

@rivetkit/react

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/react@4392

rivetkit

pnpm add https://pkg.pr.new/rivet-dev/rivet/rivetkit@4392

@rivetkit/sql-loader

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sql-loader@4392

@rivetkit/sqlite-vfs

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sqlite-vfs@4392

@rivetkit/traces

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/traces@4392

@rivetkit/workflow-engine

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/workflow-engine@4392

@rivetkit/virtual-websocket

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/virtual-websocket@4392

@rivetkit/engine-runner

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner@4392

@rivetkit/engine-runner-protocol

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner-protocol@4392

commit: aae3e6d

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@railway-app railway-app bot temporarily deployed to rivet-frontend / rivet-pr-4392 March 10, 2026 00:36 Destroyed
@NicholasKissel NicholasKissel merged commit 87cf697 into main Mar 10, 2026
14 of 17 checks passed
@NicholasKissel NicholasKissel deleted the NicholasKissel/fix-site-font branch March 10, 2026 00:39
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