Skip to content

Fix dark theme contrast for secondary text (#233)#358

Open
arya2807 wants to merge 2 commits into
imDarshanGK:mainfrom
arya2807:fix/dark-theme-contrast
Open

Fix dark theme contrast for secondary text (#233)#358
arya2807 wants to merge 2 commits into
imDarshanGK:mainfrom
arya2807:fix/dark-theme-contrast

Conversation

@arya2807

@arya2807 arya2807 commented Jun 8, 2026

Copy link
Copy Markdown

Summary

This PR improves the readability of secondary text in dark mode by increasing the contrast.

Changes Made

  • Changed text-gray-500 to text-gray-400 for the description text ("100% private · runs offline · no cloud")
  • Changed text-gray-500 to text-gray-400 for the "typing..." indicator
  • Changed text-gray-600 to text-gray-400 for the "You" label

All changes are in frontend/src/components/ChatWindow.jsx.

Related Issue

Fixes #233

Screenshots (if applicable)

Before: (text was too dark to read comfortably)
After: (text is now lighter and meets contrast standards)

Checklist

  • I have tested these changes locally in dark mode
  • The changes do not affect light mode
  • This PR addresses the issue described

Additional Notes

This is my first open source contribution – please let me know if any further changes are needed. Thank you for the opportunity!

@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

@arya2807 is attempting to deploy a commit to the Darshan's projects Team on Vercel.

A member of the Team first needs to authorize it.

@imDarshanGK

Copy link
Copy Markdown
Owner

@arya2807 Add demo

@arya2807

Copy link
Copy Markdown
Author

Thanks for the review and the suggestion to add a demo. I understand what's needed and I'm working on getting the "before" and "after" screenshots ready. Could you please give me a little time to add the demo to the PR description? I'll update it as soon as it's ready. Really appreciate your patience!
Thanks

@imDarshanGK

Copy link
Copy Markdown
Owner

@arya2807 resolve conflicts

@arya2807

Copy link
Copy Markdown
Author

@arya2807 resolve conflicts

yes , I will work on it.

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.

Fix: dark theme contrast is too low for secondary text

2 participants