feat(people): revamp people page UI with minimalist hero and centered headers#248
feat(people): revamp people page UI with minimalist hero and centered headers#248kunal-10-cloud wants to merge 2 commits into
Conversation
✅ Deploy Preview for cv-community-dashboard ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughReplaces the previous page header/timestamp on the People page with a new client-side PeopleHero component (renders "Our People", a subtitle, an avatar stack from coreTeam, and a total count badge). app/people/page.tsx is refactored to use PeopleHero and no longer tracks or displays updatedAt. Adds and exports PeopleHero from components/people, and updates TeamSection to adjust heading sizing, prefix text, descriptive paragraph, and a decorative gradient divider tied to team type. Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@app/people/page.tsx`:
- Around line 264-276: The Contributors section header and description are
left-aligned; update the element(s) under the section with id "contributors" so
the subheader and paragraph are centered to match other sections—e.g., add a
centering utility (like text-center) to the h2 (the <h2 className="text-3xl
font-bold"> element) and the paragraph (<p className="text-lg
text-muted-foreground max-w-3xl mb-6">) or to their parent container div so both
title and description render centered and keep existing spacing/styling.
🧹 Nitpick comments (1)
components/people/PeopleHero.tsx (1)
51-56: Optional: format large totals for readability.Small UX polish: if total counts grow large, formatting helps scanability.
✨ Suggested tweak
- {totalCount}+ + {totalCount.toLocaleString()}+
Atharva7126
left a comment
There was a problem hiding this comment.
looks good @kunal-10-cloud
Description
This PR enhances the People page by establishing a clear visual hierarchy and separation between the main page header and section subheaders.
Related Issue
Fixes #247
Type of change
Checklist
Screenshots (if applicable)
Before

Summary by CodeRabbit
New Features
Style
Chores
✏️ Tip: You can customize this high-level summary in your review settings.