Skip to content

refactor(ui): use next geist fonts - #1165

Merged
sadmann7 merged 1 commit into
mainfrom
sadman/next-font-geist
Oct 2, 2026
Merged

sadmann7 merged 1 commit into
mainfrom
sadman/next-font-geist

Conversation

@sadmann7

@sadmann7 sadmann7 commented Oct 2, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI balanced review requested due to automatic review settings October 2, 2026 17:19
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
tablecn Ready Ready Preview Oct 2, 2026 5:19pm UTC

Request Review

@sadmann7 sadmann7 changed the title refactor: use next geist fonts refactor(ui): use next geist fonts Oct 2, 2026
@sadmann7
sadmann7 merged commit 3604a45 into main Oct 2, 2026
8 checks passed
@sadmann7
sadmann7 deleted the sadman/next-font-geist branch October 2, 2026 17:19

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The font migration and dependency cleanup are complete and internally consistent.

Review effort: Balanced
Findings: None

What changed in this PR

Replaces the standalone Geist dependency with Next.js’s built-in Geist fonts.

Changes:

  • Configures Geist Sans and Mono through next/font/google.
  • Applies font variables at the document root.
  • Removes the obsolete geist dependency.
File Description
src/​lib/​fonts.ts Defines Next.js Geist fonts.
src/​app/​layout.tsx Applies both font variables globally.
package.json Removes the Geist package.
pnpm-lock.yaml Removes locked Geist entries.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

This branch was successfully deployed

1 active deployment
Preview — c9551b77 Deployed Oct 2, 2026 by vercel[bot]
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