Skip to content

Refactor PageContributors component for improved layout and contribut…#511

Merged
vrijraj merged 1 commit intomainfrom
dev
Apr 2, 2026
Merged

Refactor PageContributors component for improved layout and contribut…#511
vrijraj merged 1 commit intomainfrom
dev

Conversation

@vrijraj
Copy link
Copy Markdown
Contributor

@vrijraj vrijraj commented Apr 2, 2026

…or display

  • Simplified loading state with a grid layout for better visual feedback.
  • Increased the number of visible contributors from 6 to 10 for enhanced visibility.
  • Updated contributor avatar size and styling for a more polished appearance.
  • Removed unused Badge and ExternalLink components to streamline the code.

PageContributors Component Refactoring

  • Updated loading state from header + spinner to 4-column grid of pulsing placeholder circles
  • Increased visible contributors from 6 to 10
  • Refactored contributors layout to 4-column grid with circular avatar links only
  • Replaced login text display with aria-label and custom hover/focus tooltip
  • Removed Badge and ExternalLink component usage
  • Removed "+N more" count display
  • Added focus-visible ring styling to avatar links

…or display

- Simplified loading state with a grid layout for better visual feedback.
- Increased the number of visible contributors from 6 to 10 for enhanced visibility.
- Updated contributor avatar size and styling for a more polished appearance.
- Removed unused Badge and ExternalLink components to streamline the code.
@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)
site Ready Ready Preview, Comment Apr 2, 2026 1:54pm

Request Review

@vrijraj vrijraj merged commit 41c3d35 into main Apr 2, 2026
3 of 4 checks passed
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c3858b49-a796-4a95-88eb-7992f5e54e2f

📥 Commits

Reviewing files that changed from the base of the PR and between 7ff2f62 and 2d6d5bd.

📒 Files selected for processing (1)
  • components/PageContributors.tsx

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

The PageContributors.tsx component has been redesigned with a 4-column grid layout replacing the previous vertical list. The loading state now displays pulsing placeholder circles instead of a spinner. The displayed contributor limit increased from 6 to 10. Badge and external-link icons were removed in favor of a custom tooltip showing login on hover/focus, along with focus-visible ring styling.

Changes

Cohort / File(s) Summary
Contributors Component Layout Refactor
components/PageContributors.tsx
Redesigned layout from vertical list to 4-column grid, updated loading state with 6 pulsing circle placeholders, increased contributor display count from 6 to 10, removed Badge/ExternalLink icons, added custom tooltip with contributor login, added focus-visible ring styling, and removed "+N more" count display.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • amanintech
✨ 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 dev

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.

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