-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Switching theme from dark to light does not change UI #19514
Comments
I cannot repro, I think this might be caused by the dev environment, can you repro on develop or app? |
If you have hot reload enabled on a dev environment then this is a known issue. https://github.com/vector-im/element-web/blob/75b1d813f1323ecb017f36dcb1e945655d86d9a2/.env.example#L14 and #18484 |
I was in a dev environment with hot-reload, and today I can't reproduce it even there :-( |
I can reliably reproduce this on develop (ie. not a dev env, no hot reload) in chrome canary (96) but only if it's using the light theme when it first loads. Load with light theme, change to dark, change to light: remains on dark theme (hover text thinks it's in light though). Reload & it's in light again. Same whether you do it from the settings modal or the lightswitch button. |
if I toggle high contrast on&off the theme toggles start working as expected |
Hi @SimonBrandner, I am an Outreachy applicant. I am interested in solving this issue. Please can you assign it to me? Thank you |
I've assigned you to the issue. If you need any help/have any questions be sure to visit #element-dev:matrix.org! |
Awesome! I am currently working on it. |
Steps to reproduce
Outcome
What did you expect?
I expected the screen to change back to light theme.
What happened instead?
The screen did not change.
Operating system
Ubuntu 21.10
Browser information
Chromium Version 95.0.4638.54 (Official Build) snap (64-bit)
URL for webapp
develop.element.io
Application version
Latest, but also confirmed in January 2021 version.
Homeserver
matrix.org
Will you send logs?
Yes
The text was updated successfully, but these errors were encountered: