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

Switching theme from dark to light does not change UI #19514

Open
andybalaam opened this issue Oct 26, 2021 · 8 comments
Open

Switching theme from dark to light does not change UI #19514

andybalaam opened this issue Oct 26, 2021 · 8 comments
Assignees
Labels
A-Themes-Official Official themes (light, dark) A-Theming O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@andybalaam
Copy link
Contributor

Steps to reproduce

  1. Use Chromium browser
  2. Go to Settings -> Appearance
  3. Click "Dark" to choose the dark theme. Note that the screen changes colour.
  4. Click "Light". The button is selected, but the screen does not change.

theme-switch

  1. Note: if you refresh the browser, the light theme is displayed.

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

@dbkr dbkr added O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist A-Themes-Official Official themes (light, dark) A-Theming labels Oct 26, 2021
@SimonBrandner
Copy link
Contributor

I cannot repro, I think this might be caused by the dev environment, can you repro on develop or app?

@t3chguy
Copy link
Member

t3chguy commented Oct 26, 2021

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

@t3chguy t3chguy added the X-Needs-Info This issue is blocked awaiting information from the reporter label Oct 26, 2021
@andybalaam
Copy link
Contributor Author

I was in a dev environment with hot-reload, and today I can't reproduce it even there :-(

@dbkr
Copy link
Member

dbkr commented Oct 27, 2021

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.
Load with light theme, change to dark, refresh, change to light: light theme
Load with dark theme, change to light, change to dark: dark theme

Same whether you do it from the settings modal or the lightswitch button.

Confused element illustrated below:
Screenshot 2021-10-27 at 17 25 00

@SocksPls
Copy link

SocksPls commented Mar 4, 2022

if I toggle high contrast on&off the theme toggles start working as expected

@EECvision
Copy link

Hi @SimonBrandner, I am an Outreachy applicant. I am interested in solving this issue. Please can you assign it to me? Thank you

@SimonBrandner
Copy link
Contributor

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!

@EECvision
Copy link

Awesome! I am currently working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Themes-Official Official themes (light, dark) A-Theming O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

6 participants