Skip to content

fix(site): partners page#7754

Merged
mhartington merged 1 commit intomainfrom
fix-site-partners
Apr 2, 2026
Merged

fix(site): partners page#7754
mhartington merged 1 commit intomainfrom
fix-site-partners

Conversation

@mhartington
Copy link
Copy Markdown
Member

@mhartington mhartington commented Apr 2, 2026

Summary by CodeRabbit

Style

  • Improved responsive typography on the Partners page with refined headline sizing for mobile and desktop displays.
  • Enhanced visual alignment by center-aligning section headings for better visual balance.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 2, 2026

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

Project Deployment Actions Updated (UTC)
blog Building Building Preview, Comment Apr 2, 2026 3:12pm
docs Building Building Preview, Comment Apr 2, 2026 3:12pm
eclipse Ready Ready Preview, Comment Apr 2, 2026 3:12pm
site Ready Ready Preview, Comment Apr 2, 2026 3:12pm

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0f8c6d36-7c7a-4463-a38c-0406ae8b2883

📥 Commits

Reviewing files that changed from the base of the PR and between 78dbd52 and 2a3b0cc.

📒 Files selected for processing (1)
  • apps/site/src/app/partners/page.tsx

Walkthrough

The Partners page receives responsive typography adjustments. The hero heading's base size decreases from text-5xl to text-4xl with a small-screen breakpoint reintroduced (sm:text-5xl), while preserving the medium breakpoint. The secondary heading gains explicit center text alignment via a new CSS class.

Changes

Cohort / File(s) Summary
Partners Page Typography
apps/site/src/app/partners/page.tsx
Adjusted hero <h1> responsive sizing strategy—reduced base from text-5xl to text-4xl while adding sm:text-5xl breakpoint. Secondary <h2> heading now includes text-center alignment class for consistent visual hierarchy.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
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.
Title check ❓ Inconclusive The title 'fix(site): partners page' is vague and doesn't specify what was actually fixed—it merely identifies the file/page without describing the nature of the fix (typography adjustments). Consider a more descriptive title like 'fix(site): adjust partners page hero typography' or 'fix(site): center partners page subheading and refine hero sizes' to clearly communicate the specific changes made.
✅ 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.


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.

@mhartington mhartington merged commit b7a0117 into main Apr 2, 2026
15 of 17 checks passed
@mhartington mhartington deleted the fix-site-partners branch April 2, 2026 15:13
@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 2, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 2, 2026, 3:20 PM

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