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

[Bug]: Page theme isn't consistent across page reloads #1187

Open
2 tasks done
dylan-dot-c opened this issue Mar 2, 2025 · 2 comments
Open
2 tasks done

[Bug]: Page theme isn't consistent across page reloads #1187

dylan-dot-c opened this issue Mar 2, 2025 · 2 comments
Labels
bug Something isn't working

Comments

@dylan-dot-c
Copy link
Contributor

dylan-dot-c commented Mar 2, 2025

What happened?

Whenever the user chooses dark mode and the browser is refreshed, the mode returns to its original state, which is light mode.

I expect that when I change my theme to dark mode, it should remain in dark mode even when I refresh my browser.

Add screenshots

recording.webm

As you can see, when the browser reloads the current state of the theme has been set to the default light mode.

I also realize that there is already a custom hook for the localStorage but it hasn't been used when implementing this feature.

I plan to simply update it and use it to solve this issue

What browsers are you seeing the problem on?

Microsoft Edge

Code of Conduct

  • I agree to follow this project's Code of Conduct

Add screenshots

No response

What browsers are you seeing the problem on?

Microsoft Edge

Code of Conduct

  • I agree to follow this project's Code of Conduct
@dylan-dot-c dylan-dot-c added the bug Something isn't working label Mar 2, 2025
Copy link

github-actions bot commented Mar 2, 2025

Welcome, @dylan-dot-c! Your issue is like a spark that ignites innovation. We're thrilled to dive into it and work together to find a solution.

Soon the maintainers/owner will review it and provide you with feedback/suggestions.

@dylan-dot-c
Copy link
Contributor Author

Previous Issue before nextJS migration
Planning to reimpliment it and get it merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant