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

quick theme toggle not working #17590

Closed
HarHarLinks opened this issue Jun 7, 2021 · 2 comments · Fixed by matrix-org/matrix-react-sdk#6160
Closed

quick theme toggle not working #17590

HarHarLinks opened this issue Jun 7, 2021 · 2 comments · Fixed by matrix-org/matrix-react-sdk#6160
Labels
A-Themes-Official Official themes (light, dark) S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression X-Release-Blocker

Comments

@HarHarLinks
Copy link

Description

This thing
image
isn't working for me at all.

The light theme will become selected in the menu like this
image
but theme is only applied when switching via this menu.

#17581 could be related.

Steps to reproduce

  • click light/dark theme quick toggle in the top left user menu thing
  • potentially use these labs settings, especially custom themes
  • no idea since when this is, i don't normally use this feature

image

Logs being sent: yes

Version information

  • Platform: desktop

For the desktop app:

  • OS: Arch
  • Version: aur/element-desktop-nightly-bin 2021060702-1
@SimonBrandner SimonBrandner added A-Themes-Official Official themes (light, dark) S-Major Severely degrades major functionality or product features, with no satisfactory workaround X-Regression X-Release-Blocker labels Jun 7, 2021
@Zachinquarantine
Copy link

It seems, for me on Windows running element build 1.7.30, that it will only work once you close off the app completely and reopen it.

@robintown
Copy link
Member

This is because of setting watchers being broken, which is fixed in matrix-org/matrix-react-sdk#6143. However as the scope of that PR has grown, I think it might make more sense to just cherry-pick commit 40d2f8228f4d2a3f7f386d320c42ae1970932ee4 from 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) S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect X-Regression X-Release-Blocker
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants