fix: move update-native-theme into ipcRenderer useEffect (#1009)

gitify-app · Apr 16, 2024 · da2bf1c · da2bf1c
1 parent 421b9dd
commit da2bf1c
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/routes/Settings.tsx b/src/routes/Settings.tsx
@@ -62,6 +62,12 @@ export const SettingsRoute: FC = () => {
     ipcRenderer.invoke('get-app-version').then((result: string) => {
       setAppVersion(result);
     });
+
+    ipcRenderer.on('update-native-theme', (_, updatedTheme: Theme) => {
+      if (settings.theme === Theme.SYSTEM) {
+        setTheme(updatedTheme);
+      }
+    });
   }, []);
 
   useMemo(() => {
@@ -77,12 +83,6 @@ export const SettingsRoute: FC = () => {
     })();
   }, [accounts.token]);
 
-  ipcRenderer.on('update-native-theme', (_, updatedTheme: Theme) => {
-    if (settings.theme === Theme.SYSTEM) {
-      setTheme(updatedTheme);
-    }
-  });
-
   const logoutUser = useCallback(() => {
     logout();
     navigate(-1);