Skip to content

Conversation

@SameedIlyas
Copy link

@SameedIlyas SameedIlyas commented Oct 31, 2025

  • Changed Dark mode font of Perpetuity from Source Code Pro to Courier New, which is same as the light mode font.
  • Closes Issue # 229

Summary by CodeRabbit

  • Style
    • Updated default font families across multiple theme presets for consistent visual presentation.

@vercel
Copy link

vercel bot commented Oct 31, 2025

@SameedIlyas is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Oct 31, 2025

Walkthrough

The pull request updates font-family values in theme presets, replacing "Source Code Pro, monospace" with "Courier New, monospace" for all three font categories (sans, serif, mono) across the defaultPresets and perpetuity dark theme configurations.

Changes

Cohort / File(s) Summary
Theme Typography Configuration
utils/theme-presets.ts
Updated font-family values from "Source Code Pro, monospace" to "Courier New, monospace" for font-sans, font-serif, and font-mono properties in defaultPresets and perpetuity dark presets.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file modification — Only one configuration file affected
  • Repetitive changes — Same font-family replacement applied consistently across multiple preset entries
  • Configuration only — No logic, control flow, or structural changes involved

Poem

🐰 Hop, hop—new fonts take the stage!
From Source Code Pro's old page,
Courier New now marks the day,
Typography in fresh display!
Theme presets dance in monospace's way. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The PR title "fix(themes-preset): Changed Dark mode fonts of perpetuity to match light mode" accurately describes the main change in the pull request. The changeset specifically modifies the font-family values in the perpetuity dark theme preset from "Source Code Pro, monospace" to "Courier New, monospace" to align with the light mode fonts, which directly aligns with what the title conveys. The title is concise, specific, and clear enough that a teammate scanning the repository history would immediately understand that this addresses a font inconsistency between dark and light modes in the perpetuity theme.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 240f7cd and 6507054.

📒 Files selected for processing (1)
  • utils/theme-presets.ts (1 hunks)
🔇 Additional comments (1)
utils/theme-presets.ts (1)

1058-1060: LGTM! Font consistency achieved.

The change successfully fixes the font mismatch between light and dark modes in the Perpetuity theme. All three font categories now consistently use "Courier New, monospace" in both modes, matching the light mode configuration.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@SameedIlyas SameedIlyas changed the title fix(themes-preset): Changed Dark mode fonts of perpetuity to match light mode (#229) fix(themes-preset): Changed Dark mode fonts of perpetuity to match light mode Oct 31, 2025
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