Skip to content

UI Enhancement: Add Theme Toggle with Light Mode Support for Better Visualization #29#40

Open
ankitkr104 wants to merge 5 commits into
JhaSourav07:mainfrom
ankitkr104:ui-enhancement-theme-toggle-29
Open

UI Enhancement: Add Theme Toggle with Light Mode Support for Better Visualization #29#40
ankitkr104 wants to merge 5 commits into
JhaSourav07:mainfrom
ankitkr104:ui-enhancement-theme-toggle-29

Conversation

@ankitkr104
Copy link
Copy Markdown
Contributor

@ankitkr104 ankitkr104 commented Apr 20, 2026

  • Improved light mode visibility for all hero/feature/tool texts and cards
  • Applied zinc gradient to hero heading in light mode
  • Split hero heading into two lines for readability
  • Fixed alignment issues in hero section
  • Made 'Client-Side Processing' and icon black in light mode
  • Used reliable CSS/Tailwind for theme-based color switching
  • Cleaned up Hero.jsx, removed unwanted code
image

Closes #29

…isualization JhaSourav07#29

- Improved light mode visibility for all hero/feature/tool texts and cards
- Applied zinc gradient to hero heading in light mode
- Split hero heading into two lines for readability
- Fixed alignment issues in hero section
- Made 'Client-Side Processing' and icon black in light mode
- Used reliable CSS/Tailwind for theme-based color switching
- Cleaned up Hero.jsx, removed unwanted code

Closes JhaSourav07#29
@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 20, 2026

@ankitkr104 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

@ankitkr104
Copy link
Copy Markdown
Contributor Author

Hi @JhaSourav07

I put a lot of work into this PR and would really appreciate a review.

Summary of changes:

  • Improve light-mode visuals (hero, features, tools)
  • Apply zinc gradient to hero heading; split into two lines for readability
  • Fix alignment and visibility issues across cards and texts
  • Ensure Client-Side Processing is readable in light mode
  • Fix ESLint issues (unused theme, tailwind config) so CI passes

This took me most of the day — if you can take a look and bump the priority / approve / merge, I’d be very grateful. I’m happy to make any follow-up changes

thanks!

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
quick-pdf Ready Ready Preview, Comment Apr 20, 2026 0:38am

@JhaSourav07
Copy link
Copy Markdown
Owner

@ankitkr104

some texts after light mode is not visible can you make sure they are visible through out the website and also kidnly add tests for logic and make sure npm run test are passing without any errors

@ankitkr104
Copy link
Copy Markdown
Contributor Author

ankitkr104 commented Apr 20, 2026

I have tried to fix all the text are properly visible in light mode, let me know which is not visible ?

@JhaSourav07
Copy link
Copy Markdown
Owner

Screencast.From.2026-04-20.18-21-50.mp4

@ankitkr104
Copy link
Copy Markdown
Contributor Author

Hey @JhaSourav07

thanks very much for pointing out these......

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.

UI Enhancement: Add Theme Toggle with Light Mode Support for Better Visualization

2 participants