Skip to content

Club/new site#450

Open
myr124 wants to merge 21 commits into
club-site-stagingfrom
club/new-site
Open

Club/new site#450
myr124 wants to merge 21 commits into
club-site-stagingfrom
club/new-site

Conversation

@myr124

@myr124 myr124 commented May 14, 2026

Copy link
Copy Markdown
Contributor

Why

To clean up club site files so that we can start implementing Figma from design team.

What

Closes: #ISSUE_NUMBER

Doesn't close any issues but helps devs start work on their respective pages.

  • Implemented navbar and gradient backgrounds with dot pattern from figma
  • Made routes for all pages
  • Modified globals.css to include these gradients, typography, colors and more so we can finetune.

Notes:

  • I would recommend to utilize global CSS variables to avoid duplicated code and fragmented CSS.
  • The gradients I have currently implemented don't match 1:1, if you guys know how to get that data from Figma accurately to convert to CSS that would be amazing (Dylan mentioned stacked gradients)

Test Plan

Current Home Page:
image

Other page with sample text placeholder (we should remove this comp once we put actual content):
image

Mobile Nav
image

Hamburger Menu (we can add css animations imo)
image

Checklist

  • [X ] Database: No schema changes, OR I ran pnpm db:generate and committed the generated files in packages/db/drizzle/
  • [X ] Environment Variables: No environment variables changed, OR I have contacted the Development Lead to modify them on Coolify BEFORE merging.

@coderabbitai

coderabbitai Bot commented May 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 40310203-12f0-4733-a7b0-fec53f6757c7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch club/new-site

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.

@farisAwheel

Copy link
Copy Markdown
Contributor
image i do think the navbar should not contain the sign up buttons if it goes into hamburger mode

@farisAwheel farisAwheel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except the navbar behaves weirdly on mid-sized screens, see prev comment

@Adr1an04 Adr1an04 requested a review from a team as a code owner June 11, 2026 00:16
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.

3 participants