From cdca0fe9ce01b38df90fba8dbe4c8468f6195d15 Mon Sep 17 00:00:00 2001
From: Rui Figueira <rui.figueira@gmail.com>
Date: Fri, 9 Aug 2024 18:41:11 +0100
Subject: [PATCH] fix(web): use currentTheme instead of settings in toggleTheme

---
 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 5ea2f65566..c1b6e2ea07 100644
--- a/packages/web/src/theme.ts
+++ b/packages/web/src/theme.ts
@@ -39,12 +39,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);