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

Dark Mode #56

Open
anastr0 opened this issue Jan 30, 2021 · 5 comments
Open

Dark Mode #56

anastr0 opened this issue Jan 30, 2021 · 5 comments
Assignees
Labels
enhancement New feature or request UI

Comments

@anastr0
Copy link
Contributor

anastr0 commented Jan 30, 2021

Everyone needs Dark Mode!

@anastr0 anastr0 added enhancement New feature or request dependencies Difficulty: Medium help wanted Extra attention is needed UI labels Jan 30, 2021
@anandbaburajan anandbaburajan self-assigned this Jan 31, 2021
@anandbaburajan anandbaburajan removed their assignment Feb 9, 2021
@anandbaburajan anandbaburajan added gssoc21 and removed help wanted Extra attention is needed labels Mar 8, 2021
@manishprivet
Copy link

Hi, can you assign this issue to me? I would like to work in it

@anandbaburajan
Copy link
Member

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.

@manishprivet
Copy link

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

@anandbaburajan
Copy link
Member

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!

@anastr0
Copy link
Contributor Author

anastr0 commented Mar 9, 2021

@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( rocketmeet.me/poll/[pollid] ) are ssr. Which means, the page is rendered in server and the user's theme preference is stored in the client's browser. This might lead to a flash each time a poll is rendered. So make sure to handle that if you're using css variables.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI
Projects
None yet
Development

No branches or pull requests

3 participants