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

Site should stick with user's choice of light or dark mode #1462

Open
cdw9 opened this issue Nov 18, 2024 · 3 comments
Open

Site should stick with user's choice of light or dark mode #1462

cdw9 opened this issue Nov 18, 2024 · 3 comments
Assignees
Labels
Appearance Bug Something isn't working

Comments

@cdw9
Copy link
Collaborator

cdw9 commented Nov 18, 2024

I'm regularly seeing the site switch to dark mode, and sometimes an in-between version with a dark wrapper and light content. Seen in Chrome and Firefox

Steps to reproduce:

The new tab will open, and you'll stay on the libraries page. Without doing anything else, the page will switch to dark mode.

Now if you look at the tab you opened, it will be in light mode. If you open another link in a new tab, it will display light mode, but the first link you opened will display dark mode.

I haven't found a way to consistently reproduce the in-between version. The moon displays, but clicking it changes the wrapper to light mode.

Image

@cdw9 cdw9 added this to website-v2 Nov 18, 2024
@rbbeeston
Copy link
Member

This is probably also related to a similar issue where opening the search after changing the theme retains the old theme. Ultimately it's a state issue, and might be better managed on the backend rather than the front-end.

@GregKaleka, your thoughts?

@rbbeeston rbbeeston moved this to Accepted in website-v2 Nov 19, 2024
@rbbeeston rbbeeston added Bug Something isn't working Appearance labels Nov 19, 2024
@GregKaleka
Copy link
Collaborator

We could handle theming on the backend, though it would require moving the setting from localstorage to a cookie (I believe there's some sensitivity to that for anonymous users).

@GregKaleka
Copy link
Collaborator

Also FWIW, I can't reproduce this in Safari (haven't tried chrome or ff).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Appearance Bug Something isn't working
Projects
Status: Accepted
Development

No branches or pull requests

4 participants