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.
|
Thank you for the review and guidance. The requested changes have now been pushed. Please review whenever convenient. |
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)