-
Notifications
You must be signed in to change notification settings - Fork 60
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
Dark Mode #56
Comments
Hi, can you assign this issue to me? I would like to work in it |
Hey @manishprivet! Since #62 is not over yet, I think we should wait for the web design for the MVP to finish and then look into dark mode. Thanks for showing interest anyway! Also, if you're good at UI/UX, you're welcome to take up #62. |
My approach for dark mode was by using css variables, and changing those variables through a single button, all blackboxed in a single component. It doesn't require the MVP to be finished. We can assign those variables to the MVP later on too |
Sounds good, but to be open with you, the styling parts of our code don't follow best practices. We don't have a dedicated UI/UX member on our team so the CSS might be troublesome to you. So we were thinking about reworking on the CSS. Can you go through our CSS and let us know what you think? You're also suggested to join our Gitter chatroom by the way. Thanks! |
@manishprivet Thanks for your interest in contributing. You can start working on this. You can also help us in #62 if you're interested. But please note that we're using Nextjs and poll pages( Alternatively, you can make use of redux, redux-persist and somehow store theme preference in the server itself. Look into it and tell us if you have any trouble. Thanks! :D |
Everyone needs Dark Mode!
The text was updated successfully, but these errors were encountered: