Improved landing page with text visibility#591
Conversation
|
@Chandrika987 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
There was a problem hiding this comment.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
|
Hello @Priyanshu-byte-coder , |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Please fix before merging:
Hardcoded colors bypassing CSS variable system — replace all of these:
text-slate-*→text-[var(--muted-foreground)]text-white→text-[var(--foreground)]bg-white/*→bg-[var(--card-muted)]dark:*utility classes → not needed when using CSS vars (vars already handle dark mode)- Any
text-gray-*,bg-gray-*,border-gray-*→ usevar(--border),var(--muted-foreground)
All colors must use CSS vars defined in globals.css so dark/light theming works automatically.
Summary
Improved the landing page visual hierarchy, readability, and accessibility contrast by enhancing typography visibility, button distinction, spacing, and feature card styling while preserving the existing layout and functionality.
Closes #261
Type of Change
Changes Made
How to Test
Steps for the reviewer to verify this works:
npm run devScreenshots
Checklist
npm run lintpasses locallynpm run type-check)