Skip to content

fix(hackathons): single-column teams tab and primary-colored pager#562

Merged
Benjtalkshow merged 2 commits into
boundlessfi:mainfrom
Benjtalkshow:fix/teams-tab-single-column-pagination
May 15, 2026
Merged

fix(hackathons): single-column teams tab and primary-colored pager#562
Benjtalkshow merged 2 commits into
boundlessfi:mainfrom
Benjtalkshow:fix/teams-tab-single-column-pagination

Conversation

@Benjtalkshow
Copy link
Copy Markdown
Collaborator

@Benjtalkshow Benjtalkshow commented May 15, 2026

Summary

  • Revert the hackathon teams tab back to a single-column grid.
  • Rework the shared Pagination component to match the icon-chevron layout used on the organizer submissions and participants pages (first / prev / next / last buttons + "Page X of Y" indicator).
  • Style the pager buttons with the primary color (bg-primary text-primary-foreground).

Test plan

  • Open a hackathon teams tab with multiple pages of results and confirm the new pager renders.
  • Verify prev/next/first/last buttons move through pages correctly and disable at the boundaries.
  • Confirm the pager hides when there is only one page.
  • Confirm first/last buttons hide below the lg breakpoint, matching the existing organizer-page behaviour.
  • Visually compare the pager to the organizer submissions / participants pages for design consistency.

Summary by CodeRabbit

  • New Features

    • User avatars in submissions are now clickable, linking to participant profile pages in a new tab.
    • Pagination interface redesigned with directional icon buttons for improved navigation (first, previous, next, last page).
  • Style

    • Teams listing layout adjusted to single-column display format.

Review Change Stack

Revert the teams tab grid to a single column and rework the shared
Pagination component to match the icon-chevron layout used by the
organizer submissions and participants pages, styled with the primary
color.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 15, 2026

@Benjtalkshow is attempting to deploy a commit to the Threadflow Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 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: cbf218c7-e302-4453-9f6f-b94f404828c5

📥 Commits

Reviewing files that changed from the base of the PR and between f9127d8 and d14009a.

📒 Files selected for processing (4)
  • app/(landing)/hackathons/[slug]/components/tabs/contents/FindTeam.tsx
  • app/(landing)/hackathons/[slug]/components/tabs/contents/submissions/SubmissionCard.tsx
  • components/avatars/GroupAvatar.tsx
  • components/ui/pagination.tsx

Disabled knowledge base sources:

  • Linear integration is disabled

You can enable these sources in your CodeRabbit configuration.


📝 Walkthrough

Walkthrough

The PR updates the hackathon UI with three distinct changes: a new profile-linking feature for avatar components, a pagination control refactor from text-based to icon-driven navigation, and a grid layout adjustment in the team cards display from 2-column to single-column.

Changes

Hackathon UI Enhancements

Layer / File(s) Summary
GroupAvatar profile linking support
components/avatars/GroupAvatar.tsx
GroupAvatarProps extended with optional usernames array. Avatar rendering conditionally wraps each member avatar in a next/link to their profile when username is available; otherwise renders unlinked avatar.
SubmissionCard avatar profile linking integration
app/(landing)/hackathons/[slug]/components/tabs/contents/submissions/SubmissionCard.tsx
SubmissionCard header now passes usernames to GroupAvatar for team submissions. Individual participant submissions render BasicAvatar wrapped in a profile link when participant.username exists.
Pagination UI refactor to icon navigation
components/ui/pagination.tsx
Pagination replaced text/button rendering with icon-only first/previous/next/last controls. Imports updated to use lucide-react icons and shared Button component. Display changed from numeric page links to "Page X of Y" counter with directional buttons.
FindTeam grid layout adjustment
app/(landing)/hackathons/[slug]/components/tabs/contents/FindTeam.tsx
Team cards grid changed from responsive 2-column (lg:grid-cols-2) to fixed single-column layout.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

  • boundlessfi/boundless#560: Adds wiring for the Pagination control in FindTeam, complementing this PR's refactored pagination component.

Suggested reviewers

  • 0xdevcollins

Poem

🐰 A team's faces now link to their homes,
While pages turn swiftly through icon controls,
The grid stands alone, proud and tall,
Avatars gleaming with profile-page thralls! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Wrap the individual avatar on SubmissionCard in a profile link and
forward team-member usernames to GroupAvatar so each clustered avatar
opens that user's profile in a new tab.
@Benjtalkshow Benjtalkshow merged commit f33a407 into boundlessfi:main May 15, 2026
4 of 7 checks passed
Benjtalkshow added a commit that referenced this pull request May 15, 2026
) (#563)

* fix(hackathons): single-column teams tab and primary-colored pager

Revert the teams tab grid to a single column and rework the shared
Pagination component to match the icon-chevron layout used by the
organizer submissions and participants pages, styled with the primary
color.

* feat(submissions): link submission card avatars to profile pages

Wrap the individual avatar on SubmissionCard in a profile link and
forward team-member usernames to GroupAvatar so each clustered avatar
opens that user's profile in a new tab.
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