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 a dark theme #60

Open
Fleker opened this issue Jan 18, 2022 · 6 comments · Fixed by #69
Open

Add a dark theme #60

Fleker opened this issue Jan 18, 2022 · 6 comments · Fixed by #69

Comments

@Fleker
Copy link

Fleker commented Jan 18, 2022

I've enjoyed playing this game for a while, but it can definitely be a tad hard on the eyes. Adding a dark theme as an option can provide a nice UX and be consistent with other software and apps I use, including GitHub.

I can tackle this PR.

@nk9
Copy link

nk9 commented Feb 17, 2022

That would be great! I use dark mode and would love to see it for this game as well.

lanodan added a commit to lanodan/guess-my-word that referenced this issue Jan 26, 2024
hryanjones pushed a commit that referenced this issue Jan 29, 2024
@hryanjones
Copy link
Owner

@lanodan I don't suppose it's easy for you to add a toggle for folks who want to toggle it a specific way? I've had a user complain that they like some of their sites dark, but didn't want it for Guess My Word.

@hryanjones hryanjones reopened this Jan 29, 2024
@lanodan
Copy link
Contributor

lanodan commented Jan 29, 2024 via email

@Bratface23
Copy link

@lanodan I don't suppose it's easy for you to add a toggle for folks who want to toggle it a specific way? I've had a user complain that they like some of their sites dark, but didn't want it for Guess My Word.

Thanks, hryanjones! That was me, Bratface23!

@hryanjones
Copy link
Owner

Thanks for the start @lanodan ! I added onto your commit to make it save to and load from localstorage.

@hryanjones hryanjones reopened this Feb 3, 2024
@hryanjones
Copy link
Owner

I had to revert these changes as it was breaking the leaderboard page. Can revisit this with better support on all pages.

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 a pull request may close this issue.

5 participants