-
Notifications
You must be signed in to change notification settings - Fork 30
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 #11
Comments
Hey @darekkay I would love to give this a try. Is that okay? |
Hey @sarveshh , sure, feel free :) But it might not be that simple given the not so up-to-date architecture of https://github.com/darekkay/static-marks-app/ |
this would be awesome, only if I was smart enough to help :-( good luck |
Hi, @darekkay I gave it a try. For now, just implemented for theme preference, it's gonna show up according to the user theme preference in their browser settings. Looking forward to your approval to continue along. :) |
Hey @devnikesh , the work in progress looks good to me 👍 I also agree that it's good to start with the browser preferences first before implementing a theme switcher later. As a hint, have a look into the (newly added) atomic color classes that will match the CSS custom properties: https://github.com/darekkay/static-marks-app/blob/master/src/styles/_atomic.scss |
@darekkay , yeah sure thank you, glad you liked it.. 😄 |
Hey @darekkay , I'm done with the browser preference part, earlier the menu item icons and text color were not changing on dark mode, now it's fixed. Can we proceed to implement the feature ? |
Hi @darekkay, was waiting for your approval, I've done the testing and have opened a PR. Devnikesh/add dark mode 11 Here's a Screenshot of the test result: |
Version 2.4.0 published, including a Dark Mode. Thanks everyone 🎉 |
Prerequesites:
<use>
(MDN).TODO:
All changes have to be done in this repository.
The text was updated successfully, but these errors were encountered: