Skip to content

style: consolidate landing colors/spacing into theme tokens#619

Merged
ezedike-evan merged 1 commit into
ezedike-evan:mainfrom
muffti123:Theme-tokens
Jul 4, 2026
Merged

style: consolidate landing colors/spacing into theme tokens#619
ezedike-evan merged 1 commit into
ezedike-evan:mainfrom
muffti123:Theme-tokens

Conversation

@muffti123

Copy link
Copy Markdown
Contributor

Summary

Replaces all ad-hoc Tailwind color classes in landing components (Hero.tsx, StatBar.tsx) with the project-defined CSS custom property tokens, ensuring visual consistency and a single source of truth for the color palette.

Changes

app/globals.css

  • Added accent-subtle token#eff6ff (light) / #172554 (dark) — for badge/pill background fills.
  • Updated dark-mode accent from #2563eb to #93c5fd (blue‑300) so accent text/icons remain legible on dark surfaces.
  • Documented the full token set in a comment block with light/dark values and usage descriptions.

components/landing/Hero.tsx

Before After
bg-blue-50 / dark:bg-blue-950 bg-accent-subtle
text-blue-700 / dark:text-blue-300 text-accent
text-gray-900 / dark:text-white text-primary-text
text-blue-600 text-accent
text-gray-600 / dark:text-gray-400 text-secondary-text

components/landing/StatBar.tsx

Before After
border-gray-200 / dark:border-gray-700 border-border
bg-gray-50 / dark:bg-gray-800/50 bg-bg-subtle
text-blue-600 text-accent
text-gray-900 / dark:text-white text-primary-text
text-gray-500 / dark:text-gray-400 text-secondary-text

Acceptance criteria

  • No hardcoded hex or ad-hoc color classes in landing components
  • All colors reference project theme tokens
  • Token set documented in globals.css

Closes #531

@drips-wave

drips-wave Bot commented Jun 28, 2026

Copy link
Copy Markdown

@muffti123 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@ezedike-evan ezedike-evan left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Two things to fix before this can merge:

  1. Commit message — the subject Consolidate landing colors/spacing into theme tokens doesn't follow Conventional Commits. Please use a type prefix, e.g. style(landing): consolidate colors/spacing into theme tokens and push.

  2. CI not triggered — no checks are running on this branch. Please push an empty commit or close and reopen to trigger GitHub Actions.

@muffti123

muffti123 commented Jun 28, 2026 via email

Copy link
Copy Markdown
Contributor Author

…#531)

Replace ad-hoc Tailwind color classes in Hero, StatBar, and the
StatBar icon colors in app/page.tsx with the project's CSS custom
property tokens (accent, accent-subtle, primary-text, secondary-text,
border, bg-subtle). Adds accent-subtle and documents the full token
set in globals.css.
@vercel

vercel Bot commented Jul 4, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the ezedikeevan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ezedike-evan ezedike-evan changed the title Consolidate landing colors/spacing into theme tokens style: consolidate landing colors/spacing into theme tokens Jul 4, 2026
@ezedike-evan ezedike-evan merged commit fc4f7b0 into ezedike-evan:main Jul 4, 2026
12 of 17 checks passed
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.

B098: [ENHANCEMENT] [UI] Theme tokens pass for landing

2 participants