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 #4

Open
sagar-barapatre opened this issue Feb 6, 2021 · 22 comments
Open

Dark Mode #4

sagar-barapatre opened this issue Feb 6, 2021 · 22 comments
Labels
enhancement New feature or request Level2 hard UIs/ReactJS requires knowledge of React.JS

Comments

@sagar-barapatre
Copy link
Contributor

It's not required now but in future we can implement the feature of dark mode on our website. So that user to toggle the button to switch to different themes.

@Namanl2001
Copy link
Owner

Thanks! again @sagar-barapatre for opening this issue.
In general, I see a dark theme as a nice-to-have, but we have to weigh the costs. My main reservations are:

  • It will be more work to implement than it may seem at first. CSS styles are in inline styles in the JSX.
  • We also need to make sure that everything still looks ok in both light and dark modes after any new addition or change.
  • visual testing is more difficult.

None of that is a major problem, if someone wants to contribute to this, I'm happy to help.

@Namanl2001 Namanl2001 added Level2 hard enhancement New feature or request UIs/ReactJS requires knowledge of React.JS labels Feb 18, 2021
@sagar-barapatre
Copy link
Contributor Author

Yeah! I think it's a big task, so let this issue be open. This issue will maybe require a team of 3-4 people who can review over the feature.

@thaditi
Copy link

thaditi commented Feb 23, 2021

Hey @Namanl2001 and @sagar-barapatre. I would like to work on this issue as part of GSSOC'21.

@khareyash05
Copy link
Contributor

Hey @Namanl2001 and @sagar-barapatre .I would like to work on this issue.If no one else is assigned.I will work on this issue as a part of GSSOC'21

@sagar-barapatre
Copy link
Contributor Author

Actually this issue might require 2-3 people for testing this feature. So shall we together work on this issue ?
What say @Namanl2001 @thaditi @khareyash05 ??

@khareyash05
Copy link
Contributor

@sagar-barapatre I would love to

@thaditi
Copy link

thaditi commented Feb 26, 2021

@sagar-barapatre Sure.

@Namanl2001
Copy link
Owner

Great! now as we have a team of three I think you all can start discussing your ideas and approach with each other. Here on the issue or in the discussions https://github.com/Namanl2001/MERN-Gurujii-dev/discussions.

Hopefully we would have a separate discord channel for our project and a project mentor with us soon, provided by the GSSoC'21 team.
Thank you @thaditi @khareyash05 @sagar-barapatre for taking up this issue.

All the best!!

@sagar-barapatre
Copy link
Contributor Author

Great! now as we have a team of three I think you all can start discussing your ideas and approach with each other. Here on the issue or in the discussions https://github.com/Namanl2001/MERN-Gurujii-dev/discussions.

Hopefully we would have a separate discord channel for our project and a project mentor with us soon, provided by the GSSoC'21 team.
Thank you @thaditi @khareyash05 @sagar-barapatre for taking up this issue.

All the best!!

P.S. - I m already a mentor

@khareyash05
Copy link
Contributor

I have added a dark mode toggle. on the bottom right corner.Its a perfect working one and we can also isolate those elements whose styling is not to be changed....Also it remembers the mode(dark/light) when the last time u opened it.
Suggest me with some changes
2021-03-07

@Namanl2001
Copy link
Owner

@khareyash05 open a PR then it would be easy to review it.

@khareyash05
Copy link
Contributor

I have opened a PR kindly review the same

@khareyash05 khareyash05 mentioned this issue Mar 8, 2021
@sagar-barapatre
Copy link
Contributor Author

Can you place the toggle button just besides the Google signin button, coz generally dark mode buttons are place at top right corners in almost every website.
This is just a suggestion.

@Namanl2001 Namanl2001 added the GSSOC21 Girlscript summer of code 2021 label Mar 9, 2021
@ritik307
Copy link

@sagar-barapatre @Namanl2001 @khareyash05 is anyone working on this issue. If not then can you plz assign this issue to me?

@khareyash05
Copy link
Contributor

@ritik307 I am working on the issue

@Jassi10000-zz
Copy link

@Namanl2001 Is the issue still open , being a GSSOC'21 participant I want to try my hands out on this issue

@sagar-barapatre
Copy link
Contributor Author

@Jassi10000 PR has already been made. So you can suggest some changes which you would like to if you want.

@Jassi10000-zz
Copy link

Sure

@khareyash05
Copy link
Contributor

khareyash05 commented Mar 31, 2021

2021-03-31 (7)
@sagar-barapatre @Namanl2001 I am getting this error Kindly help me
I have not changed a thing in the file mentioned in the error

@sagar-barapatre
Copy link
Contributor Author

sagar-barapatre commented Mar 31, 2021

set { hidden : false }
The error gets removed

@subhangi2731
Copy link

@Namanl2001 @sagar-barapatre please assign me this I want to work on this

@sagar-barapatre
Copy link
Contributor Author

Okay assigned.

@Namanl2001 Namanl2001 removed the GSSOC21 Girlscript summer of code 2021 label Oct 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Level2 hard UIs/ReactJS requires knowledge of React.JS
Projects
None yet
Development

No branches or pull requests

7 participants