Skip to content

chore(site): layout updates#7786

Merged
mhartington merged 1 commit intomainfrom
site-updates
Apr 8, 2026
Merged

chore(site): layout updates#7786
mhartington merged 1 commit intomainfrom
site-updates

Conversation

@mhartington
Copy link
Copy Markdown
Member

@mhartington mhartington commented Apr 8, 2026

Summary by CodeRabbit

  • Style
    • Adjusted layout spacing and padding across homepage, product pages, and pricing sections
    • Improved responsive design with better mobile and small screen alignment
    • Updated typography sizing and button styling for visual consistency
    • Enhanced navigation component implementation

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

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

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 8, 2026 1:11am
docs Ready Ready Preview, Comment Apr 8, 2026 1:11am
eclipse Ready Ready Preview, Comment Apr 8, 2026 1:11am
site Ready Ready Preview, Comment Apr 8, 2026 1:11am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 8, 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: 234c72a5-2216-4611-bed9-cf39089604ad

📥 Commits

Reviewing files that changed from the base of the PR and between 15b15e2 and 7feec75.

📒 Files selected for processing (6)
  • apps/site/src/app/(index)/page.tsx
  • apps/site/src/app/orm/page.tsx
  • apps/site/src/app/postgres/page.tsx
  • apps/site/src/app/pricing/pricing-hero-plans.tsx
  • apps/site/src/app/studio/page.tsx
  • apps/site/src/components/homepage/card-section/card-section.tsx

Walkthrough

The changes adjust responsive layouts and spacing across multiple pages and components in the site app. Modifications include replacing inline anchor tags with Next.js Link components, removing vertical spacing classes from content containers, tweaking responsive alignment and padding on hero sections and card grids, and restructuring the Studio page hero with updated typography styling.

Changes

Cohort / File(s) Summary
Spacing & Padding Normalization
apps/site/src/app/(index)/page.tsx, apps/site/src/app/postgres/page.tsx, apps/site/src/app/pricing/pricing-hero-plans.tsx
Removed my-12 py-12 vertical spacing classes from content containers; adjusted pt-48 to pt-40 on hero sections; updated section wrapper classes with hero and content designations. Also normalized whitespace in pricing text.
ORM Page Refactoring
apps/site/src/app/orm/page.tsx
Added Link import and replaced anchor tags with <Link> components; implemented responsive layout patterns across multiple sections (badge/footer centering on small screens, stats row switching to column layout below breakpoint, card content centering with left-align on medium+); added responsive margin behavior on CTA buttons.
Studio Page Hero Restructuring
apps/site/src/app/studio/page.tsx
Removed top negative margin (-mt-24) from main container; restructured hero section with updated padding (pt-40), flexbox alignment (flex items-end justify-center), and gradient background; refactored eyebrow text from styled paragraph to flex row with span markup; updated hero description sizing (max-w-2xl) and CTA buttons with larger size variant (3xl) and explicit font styling overrides; added responsive image sizing classes.
Card Section Component Padding
apps/site/src/components/homepage/card-section/card-section.tsx
Adjusted horizontal padding: sectionClass now includes px-4 sm:px-0 for responsive inset; outer wrapper changed from px-4 to px-0 sm:px-4, altering mobile vs. desktop padding behavior.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 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 'chore(site): layout updates' is vague and generic, using the non-descriptive term 'layout updates' without specifying which layouts or the nature of the changes. Consider a more specific title like 'chore(site): adjust spacing and responsive alignment across pages' that better conveys the scope of the layout modifications being 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.

@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 8, 2026

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

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 8, 2026, 1:17 AM

@mhartington mhartington merged commit 786ece4 into main Apr 8, 2026
17 of 18 checks passed
@mhartington mhartington deleted the site-updates branch April 8, 2026 01:26
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