feat(ui): enhance landing page styling and responsiveness#588
feat(ui): enhance landing page styling and responsiveness#588BCA-krishna wants to merge 2 commits into
Conversation
|
@BCA-krishna 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!
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Two critical issues:
-
Missing EOF newlines — multiple files end without a trailing newline.
-
Hardcoded Tailwind colors break theming — the redesign uses
from-slate-950 via-slate-900 to-slate-950,text-white,text-slate-300, etc. These bypass the theme system and will look wrong in light mode. All colors must go through CSS variables:- Background: use
var(--background) - Text: use
var(--foreground),var(--muted-foreground) - Avoid all raw
slate-*,white,blackTailwind classes — use CSS vars instead.
- Background: use
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Hardcoded colors must be replaced with CSS vars:
- Missing EOF newline
- Raw Tailwind/hardcoded colors:
text-white→text-[var(--foreground)]text-slate-300,text-slate-900→text-[var(--muted-foreground)]/text-[var(--foreground)]bg-white,bg-white/5,bg-white/10,border-white/20,hover:border-white→bg-[var(--background)],bg-[var(--card-muted)],border-[var(--border)]hover:bg-slate-100→hover:bg-[var(--control-hover)]group-hover:text-blue-300→group-hover:text-[var(--accent)]
CSS vars in globals.css handle dark/light automatically — no need for raw color classes.
|
@Priyanshu-byte-coder Resolved the requested review changes:
All requested changes have been pushed to the PR branch. |
Description
Enhanced the landing page UI with modern styling improvements and better responsiveness.
Changes Made
Screenshots
Before
After
Closes #581