Skip to content

Update the typography Jetbrains to Inter#110

Merged
Ayush8923 merged 1 commit intomainfrom
fix/update-typography
Apr 8, 2026
Merged

Update the typography Jetbrains to Inter#110
Ayush8923 merged 1 commit intomainfrom
fix/update-typography

Conversation

@Ayush8923
Copy link
Copy Markdown
Collaborator

@Ayush8923 Ayush8923 commented Apr 8, 2026

Summary:

  • As per the discussion, the Jetbrains font not perfectly fitting in the body, so updating this with Inter.

Summary by CodeRabbit

  • Style
    • Updated the default font to Inter for improved consistency across the application.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 22f2fa04-31f2-4b19-b42e-cceeb0288d5d

📥 Commits

Reviewing files that changed from the base of the PR and between 25ab018 and e3ac849.

📒 Files selected for processing (1)
  • app/layout.tsx

📝 Walkthrough

Walkthrough

The pull request updates the font configuration in the application layout, replacing the Plus Jakarta Sans font with Inter. The font variable reference in the body className is updated accordingly, while other font settings remain unchanged.

Changes

Cohort / File(s) Summary
Font Configuration
app/layout.tsx
Replaced Plus_Jakarta_Sans import with Inter font; updated body className variable from jakarta.variable to inter.variable. JetBrains_Mono configuration and usage unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

ready-for-review

Suggested reviewers

  • AkhileshNegi
  • nishika26
  • Prajna1999

Poem

🐰 hop hop
From Jakarta Sans we bid adieu,
Inter fonts now fresh and new,
Hop by hop, the style's refined,
Better kerning, hearts aligned! 💕

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title claims to update typography from Jetbrains to Inter, but the actual change only replaces Plus_Jakarta_Sans with Inter while keeping JetBrains_Mono unchanged. Revise the title to accurately reflect that only the body font (Plus_Jakarta_Sans) is being replaced with Inter, or clarify which Jetbrains font is being updated.
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/update-typography

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.

@Ayush8923 Ayush8923 requested a review from vprashrex April 8, 2026 08:43
@Ayush8923 Ayush8923 self-assigned this Apr 8, 2026
@Ayush8923 Ayush8923 requested a review from AkhileshNegi April 8, 2026 08:43
@Ayush8923 Ayush8923 merged commit 77079eb into main Apr 8, 2026
2 checks passed
@Ayush8923 Ayush8923 deleted the fix/update-typography branch April 8, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants