Noteslify icon indicating copy to clipboard operation
Noteslify copied to clipboard

Add Theming Option to the Notes and Homepage/everywhere else

Open devarshishimpi2 opened this issue 2 years ago • 8 comments

Add the options of switching themes from light to dark theme with a toggle

devarshishimpi2 avatar Sep 16 '22 18:09 devarshishimpi2

I can do that. Let me do this as well.

khushi818 avatar Sep 17 '22 16:09 khushi818

I can do that. Let me do this as well.

Okay, have assigned you. But were you able to self-assign yourself as you have write access?

devarshishimpi avatar Sep 17 '22 16:09 devarshishimpi

Oh do I ? I didn't check

khushi818 avatar Sep 17 '22 17:09 khushi818

Yup I have it

khushi818 avatar Sep 17 '22 17:09 khushi818

Partially added in https://github.com/dvstechlabs/Noteslify/pull/47

devarshishimpi avatar Sep 24 '22 11:09 devarshishimpi

Partial Fix in https://github.com/dvstechlabs/Noteslify/pull/89

devarshishimpi avatar Oct 09 '22 06:10 devarshishimpi

i can add the dark mode and light mode feature

krishnabiradar310802 avatar Oct 15 '22 17:10 krishnabiradar310802

Assigned

devarshishimpi avatar Oct 15 '22 17:10 devarshishimpi

I'd like to work on implementing dark mode for all pages, please assign me. Or do I need to open a new issue?

nkirukka avatar Nov 27 '22 15:11 nkirukka

I'd like to work on implementing dark mode for all pages, please assign me. Or do I need to open a new issue?

@nkirukka Hi, This issue is open and have assigned to you.

We are shifting to using Tailwind in Frontend CSS. ( Dashboard is not using Tailwind CSS, it's just the new landing page being developed for now ) You can help us out here as well 😄 https://github.com/dvstechlabs/Noteslify/issues/237

devarshishimpi avatar Nov 27 '22 15:11 devarshishimpi

  1. Which of the folders should I work in? /frontend or /frontendnew?
  2. The page that opens on localhost:3000 from /frontend is different from the current live site and different from /frontendnew.
  3. Is there a color guide for dark theme or do I have creative liberty?

See attactment for better understanding

Current live

live

From /frontendnew

frontendnew

From /frontend

The navbar here specifically is different frontend

nkirukka avatar Nov 27 '22 20:11 nkirukka

@nkirukka

Which of the folders should I work in? /frontend or /frontendnew?

/frontendnew

The page that opens on localhost:3000 from /frontend is different from the current live site and different from /frontendnew.

Yes, since it's not running the latest version due to some under-development features like the folders section. The older frontend landing page is no longer developed. ( Only the landing page ) The new production version once released will have the new landing page, all previous landing pages will be archived. image

Is there a color guide for dark theme or do I have creative liberty?

Nothing specific for dark theme. Feel free to use what you think the best!!!

devarshishimpi avatar Nov 28 '22 11:11 devarshishimpi

@nkirukka You can start working on it 🎉

Also, Could you make it in such a way that only adding the prefix dark: to any style element selects the dark theme for that element, for example, I wanted to make a title white in the dark theme and black in light theme, so I should be able to use <p className="text-black dark:text-white">Some Text</p> Hope you get my point! And then you can continue to implement dark theme on all the pages!

devarshishimpi avatar Jan 14 '23 17:01 devarshishimpi

@nkirukka Updates?

devarshishimpi avatar Jan 24 '23 11:01 devarshishimpi

Can I start working on this?

thevinitgupta avatar Jan 27 '23 13:01 thevinitgupta

Can I start working on this?

Sure @thevinitgupta Have assigned you the same.

devarshishimpi avatar Jan 27 '23 14:01 devarshishimpi

Also, Could you make it in such a way that only adding the prefix dark: to any style element selects the dark theme for that element, for example, I wanted to make a title white in the dark theme and black in light theme, so I should be able to use <p className="text-black dark:text-white">Some Text</p>

@devarshishimpi I have configured tailwind to do this as you asked. But the pages accessible (like Pricing, Features) already have the dark mode options While the Signup and Contact pages don't have any content.

Could you please clarify?

thevinitgupta avatar Jan 28 '23 13:01 thevinitgupta

Also, Could you make it in such a way that only adding the prefix dark: to any style element selects the dark theme for that element, for example, I wanted to make a title white in the dark theme and black in light theme, so I should be able to use <p className="text-black dark:text-white">Some Text</p>

@devarshishimpi I have configured tailwind to do this as you asked. But the pages accessible (like Pricing, Features) already have the dark mode options While the Signup and Contact pages don't have any content.

Could you please clarify?

Yes, that's great. You can raise a PR for the same. Sign-up and Contact Pages haven't been made yet.

devarshishimpi avatar Jan 28 '23 15:01 devarshishimpi

Fixed! Thanks to @thevinitgupta and @nkirukka 🎉

devarshishimpi avatar Mar 20 '23 18:03 devarshishimpi