From 3d69c591d3599cc789fd20c05bce3baba890b157 Mon Sep 17 00:00:00 2001 From: Rui Figueira Date: Mon, 12 Aug 2024 21:19:30 +0100 Subject: [PATCH] fix(web): use currentTheme instead of settings in toggleTheme (#32104) --- packages/web/src/theme.ts | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/packages/web/src/theme.ts b/packages/web/src/theme.ts index 5534e73646..b12fc0b7db 100644 --- a/packages/web/src/theme.ts +++ b/packages/web/src/theme.ts @@ -45,12 +45,8 @@ type Theme = 'dark-mode' | 'light-mode'; const listeners = new Set<(theme: Theme) => void>(); export function toggleTheme() { - const oldTheme = settings.getString('theme', 'light-mode'); - let newTheme: Theme; - if (oldTheme === 'dark-mode') - newTheme = 'light-mode'; - else - newTheme = 'dark-mode'; + const oldTheme = currentTheme(); + const newTheme = oldTheme === 'dark-mode' ? 'light-mode' : 'dark-mode'; if (oldTheme) document.body.classList.remove(oldTheme);