feat(web): add smooth theme transition animation#536
Conversation
|
@Jai-Shankar1033 is attempting to deploy a commit to the Prashantkumar Khatri's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Hi @Jai-Shankar1033, Thanks for opening this pull request. This PR has been automatically classified based on the files modified. Applied Labels
Primary Review Area
Reviewer@ShantKhatri has been identified as the primary reviewer for this pull request. If you have any questions regarding the affected area or implementation details, feel free to reach out to the assigned reviewer. Thank you for your contribution! |
CI — Checks FailedBackend — SKIP
Mobile — SKIP
Web — FAIL
Last updated: |
|
Hey @ShantKhatri, just a heads up — the web-ci lint errors showing up are pre-existing in the codebase and weren't introduced by this PR. I checked the changes and all 4 modified files are clean. Let me know if you have any feedback on the implementation! |
|
Congratulations @Jai-Shankar1033 on getting PR #536 merged! Thank you for your contribution to the project. To receive the appropriate GSSoC labels and recognition, please mention @Harxhit in the #get-labels channel on our Discord server and share your merged PR link. |
* fix: improve feature card readability and contrast (Dev-Card#319) * feat(web): add smooth theme transition animation
* fix: improve feature card readability and contrast (Dev-Card#319) * feat(web): add smooth theme transition animation
Summary
The theme toggle was switching instantly which felt jarring compared to the rest of the UI. This PR adds smooth transitions so backgrounds, text, borders and shadows all fade gracefully when switching themes. Also polished up the toggle icon so it animates out and back in instead of just swapping abruptly.
Closes #321
Type of Change
What Changed
How to Test
Checklist
pnpm -r run lintpasses).pnpm -r run typecheck).pnpm -r run test).console.logor debug statements left in the code.Screenshots / Recordings
Additional Context
Checked performance after the changes — no regression:
LCP 0.12s, CLS 0.01, INP 24ms all in the good range.