Skip to content

perf: remove unused root layout font initialization#43

Merged
charlesrhoward merged 1 commit intomainfrom
codex/feat-31-remove-unused-layout-font-init
Feb 21, 2026
Merged

perf: remove unused root layout font initialization#43
charlesrhoward merged 1 commit intomainfrom
codex/feat-31-remove-unused-layout-font-init

Conversation

@charlesrhoward
Copy link
Contributor

@charlesrhoward charlesrhoward commented Feb 21, 2026

Summary

  • remove unused Geist and Geist_Mono initialization from the root layout
  • keep existing typography and styling behavior unchanged (font-sans remains in use)

Closes #31


Note

Low Risk
Small cleanup limited to removing unused font code in the root layout; minimal chance of visual regressions since the variables were not referenced.

Overview
Removes the unused Geist/Geist_Mono imports and initialization from app/layout.tsx, leaving the layout to rely on existing global styling (e.g., font-sans) without changing runtime behavior.

Written by Cursor Bugbot for commit ff5051f. This will update automatically on new commits. Configure here.

@charlesrhoward charlesrhoward enabled auto-merge (squash) February 21, 2026 21:29
@vercel
Copy link

vercel bot commented Feb 21, 2026

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

Project Deployment Actions Updated (UTC)
drawit Ready Ready Preview, Comment Feb 21, 2026 10:17pm

Request Review

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a17d357697

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

import type React from "react"
import type { Metadata, Viewport } from "next"
import { Geist, Geist_Mono } from "next/font/google"
import { Analytics } from "@vercel/analytics/next"

Choose a reason for hiding this comment

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

P2 Badge Keep Geist font registration in root layout

Removing the Geist/Geist_Mono initialization also removes the only next/font registration in the app, so the font-sans stack now points at "Geist"/"Geist Fallback" names that are never defined (see app/globals.css lines 397-398). In environments where Geist is not locally installed, text will silently fall back to a different system font, so this change does alter typography despite being labeled as no-op.

Useful? React with 👍 / 👎.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is ON. A Cloud Agent has been kicked off to fix the reported issue.

@cursor
Copy link

cursor bot commented Feb 21, 2026

Bugbot Autofix prepared fixes for 1 of the 1 bugs found in the latest run.

  • ✅ Fixed: Removing font loader may break Geist font loading
    • Reintroduced next/font/google Geist and Geist_Mono loaders in the root layout and wired Tailwind font tokens to their CSS variables so Geist fonts are actually loaded and applied.

@charlesrhoward charlesrhoward force-pushed the codex/feat-31-remove-unused-layout-font-init branch from 623e1a2 to ff5051f Compare February 21, 2026 22:16
@charlesrhoward charlesrhoward merged commit a7a551d into main Feb 21, 2026
4 checks passed
@charlesrhoward charlesrhoward deleted the codex/feat-31-remove-unused-layout-font-init branch February 21, 2026 22:18
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.

Performance: Remove unused font initialization from root layout

1 participant