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

Add dark mode support to website #38

Merged
merged 6 commits into from
Feb 28, 2025
Merged

Add dark mode support to website #38

merged 6 commits into from
Feb 28, 2025

Conversation

LouisDISPA
Copy link
Contributor

Hello, I quickly did a dark mode to the website just for fun and see how complex it would be. This is a very simple implementation only in css. Feel free to close this PR if you don't want this feature or don't like it.

I'm happy to work more on it if you are interested in this feature like:

  • cleanup CSS variables
  • add a theme switcher button (would require some javascript)
  • test different colors

List of changes:

  • Use CSS prefers-color-scheme media queries
  • Create dark version of the logo for dark mode
  • Code block syntax highlighting
  • Syntax highlighting dark mode support
  • Update Zola to v0.20
  • Add hover state for links

Screenshots:

Capture d’écran 2025-02-17 à 22 09 41 Capture d’écran 2025-02-17 à 22 09 50 Capture d’écran 2025-02-17 à 22 10 05 Capture d’écran 2025-02-17 à 22 10 23 Capture d’écran 2025-02-17 à 22 10 46

@sylvestre
Copy link
Collaborator

this is excellent, thanks!

@tertsdiepraam
Copy link
Member

Looks excellent! I like the implementation with CSS variables too. Any reason to keep this as a draft?

@LouisDISPA
Copy link
Contributor Author

Sorry for the late response.
I put this as a draft to see if you were interested in this.
Thanks you very much for the feedback!

I fixed up a bit the variable and also propose to capitalize the menu and titles for consistency.
I can revert the last commit if you don't like it.

Capture d’écran 2025-02-27 à 23 05 59

I tested all pages in light and dark mode, I think it's safe now.
You can merge this PR like this it !

@LouisDISPA LouisDISPA marked this pull request as ready for review February 27, 2025 22:09
@sylvestre sylvestre merged commit aa24730 into uutils:main Feb 28, 2025
2 checks passed
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.

3 participants