Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking β€œSign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Dark Mode Support Across UI Components #244

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sarmadhamdani02
Copy link

πŸ”₯ Description

This PR adds full dark mode support across the UI, ensuring consistent styling for text, backgrounds, and borders.

πŸ› οΈ Changes Made

  • Updated all UI components to support dark mode.
  • Applied:
    • Text Color: text-[#FFFDF8]
    • Backgrounds: bg-gray-900
    • Borders: border-gray-700
  • Ensured buttons, inputs, dropdowns, and interactive elements adapt to dark mode.
  • Fixed Tailwind dark mode behavior by setting darkMode: 'class'.

🎨 Screenshots (Before & After)

Before After (Dark Mode)
🌞 Light Mode UI πŸŒ™ Fully Themed Dark Mode UI
image image

βœ… How to Test

  1. Run the project locally.
  2. Click the dark mode toggle in the navbar.
  3. Ensure text, backgrounds, and borders correctly switch between light and dark mode.
  4. Refresh the page β†’ Dark mode should persist.

⚠️ Checklist

  • UI elements follow dark mode styling.
  • Toggle button correctly switches themes.
  • Dark mode setting persists across refreshes.

🀝 Additional Notes

  • Let me know if any components still need adjustments.
  • Future improvements: Add animations for theme transitions.

πŸš€ Ready for review & merge!

Sorry, something went wrong.

@cyclotruc
Copy link
Owner

@sarmadhamdani02 Hi

thanks a lot for your contribution

It looks good so far, one minor thing I caught was this:
image

It seems like this the text fields in the form need some adjustments to be properly displayed in darkmode

Let me know if you need any help

@sarmadhamdani02
Copy link
Author

@cyclotruc
Thanks for reviewing the PR and figuring out the problem. I'll be fixing this ASAP.

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.

None yet

2 participants