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

style: add dark mode 💄 #515

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

flixlix
Copy link

@flixlix flixlix commented Jul 26, 2023

Add Dark Mode 💄

Note: this PR is dependent on #514

Not sure if this is the most efficient implementation of a dark mode toggle.

Another Issue I unfortunately don't know how to fix is, saving this choice in local storage, would love to get some guidance on how to achieve that.

The theme automatically updates in case the system preferences change before or during using the page. Furthermore, there is a toggle if, for some reason, the theme doesn't match the user's preferences.

Here are some screenshots of how this looks like:

Toolbar in Light Mode:

Light Mode Toolbar

Toolbar in Dark Mode:

Dark Mode Toolbar

Tooltip on hover:

Tooltip

Behaviour:

Bildschirmaufnahme.2023-07-26.um.19.33.27.mov

Light Mode:

Light Mode

Dark Mode:

Dark Mode

Restyled the overall design, loosened up on some drop-shadows, corrected some spacings on the header/ footer and created new design for the device card

The changes were mainly applied to the main dashboard. Any subsequent pages have not been affected
Not sure if this is the most efficient implementation of a dark mode toggle.

Another Issue I unfortunately don't know how to fix is, saving this choice in local storage, would love to get some guidance on how to achieve that
Not sure if this is the most efficient implementation of a dark mode toggle.

Another Issue I unfortunately don't know how to fix is, saving this choice in local storage, would love to get some guidance on how to achieve that
@randybb
Copy link

randybb commented Jul 26, 2023

Nice, but please without that toggle button. Mode has to be switched via browser settings (discord thread)

@flixlix
Copy link
Author

flixlix commented Jul 26, 2023

Nice, but please without that toggle button. Mode has to be switched via browser settings (discord thread)

Removed the toggle. Thanks for pointing it out 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants