ufosc / SwampScheduler

The future of scheduling is here.
https://osc.rconde.xyz
GNU Affero General Public License v3.0
22 stars 43 forks source link

Dark Mode 🌚 #56

Open RobertConde opened 9 months ago

RobertConde commented 9 months ago

Come, join the dark side!

saltedtea commented 9 months ago

Can I join the dark side?

RobertConde commented 9 months ago

Can I join the dark side?

yess!

saltedtea commented 9 months ago

Is there a way for me to obtain a CORS API key?

saltedtea commented 9 months ago

Nevermind, I got it.

RobertConde commented 9 months ago

Okok, make sure to rename it to .env :)

saltedtea commented 9 months ago

Got this atm image

saltedtea commented 9 months ago

Any suggestions? image

RobertConde commented 9 months ago

That looks really great! I would look at Tailwind CSS's dark mode functionality to implement this correctly (e.g. dark:{classname}).

Here are some of my thoughts on the theming (this is completely subjective):

This is great work! Experiment a bit, and see if it works. Look around for great examples!

saltedtea commented 9 months ago

Thanks, I will look into that and get back to you.

saltedtea commented 9 months ago

I am back with the changes and this is what it looks like now. You can check the changes I made here: https://github.com/saltedtea/SwampScheduler

image

saltedtea commented 9 months ago

Had to take care of some personal stuff so I was only able to make changes recently.

saltedtea commented 9 months ago

Let me know if I can improve anything and if I can help make this mobile responsive.

saltedtea commented 8 months ago

@RobertConde can you review pull request #59

forrester-aidan commented 5 months ago

Hey @RobertConde, I just completed some work on the dark mode functionality as well and took your advice. I went back and got used to the proper Tailwind implementation, and here's what I have.

image

Here's a little justification for my work:

I know it's a lot to read, but let me know what you think!

Anthony42540 commented 5 months ago

@RobertConde Just wanted to show my implementation of dark mode for SwampScheduler, but I see @forrester-aidan already showed his finished work. The light/dark mode depends on the user's OS settings. image

RobertConde commented 5 months ago

@RobertConde Just wanted to show my implementation of dark mode for SwampScheduler, but I see @forrester-aidan already showed his finished work. The light/dark mode depends on the user's OS settings. image

I really like the blue!!!

I don't enjoy the colors in the schedule. Could you try to change them? I think I have an array of colors somewhere in the code. You could change them depending on which mode the user has selected.

RobertConde commented 5 months ago

Hey @RobertConde, I just completed some work on the dark mode functionality as well and took your advice. I went back and got used to the proper Tailwind implementation, and here's what I have.

image

Here's a little justification for my work:

  • I used the template dark mode that you sent me as a reference to complete this portion, so those were the decisions behind the purple buttons. On hover, they change to a slightly darker shade of purple, indicating that they're being hovered over. I also emboldened the font to make it easier to see.
  • In order to make it so none of the elements blended into the background, I changed the background of the whole page in dark mode only to the darkest color. That way, all other elements will sit above it.
  • Like you mentioned, the course and section elements are logically different, so I made the section elements a slightly lighter shade of grey to distinguish the difference between the two.
  • I liked keeping the original colors in the schedule generator from the light mode, as they're still calming colors that are easy on the eyes. In dark mode, I just created a white border for all the schedule boxes, but kept the text color the same.
  • I couldn't decide between changing the background color of the search text box or not, and I ultimately left it the same as it is in light mode. I believe it provides greater emphasis to the user that they need to be interacting with THIS text box to select the classes they need.
  • Finally, I made sure the section colors and the course Droppable element were different colors, that way it was easier to see which courses were placed in which box.

I know it's a lot to read, but let me know what you think!

That's really great!

I do think we should stay away from using pure black though and I'm not sure if purple fits in with the light mode.