refactor(hooks): resolve pr comments

nextui-org · Oct 3, 2024 · a87ad5f · a87ad5f
1 parent fdc1f92
commit a87ad5f
Showing 1 changed file with 13 additions and 15 deletions.
diff --git a/packages/hooks/use-theme/src/index.ts b/packages/hooks/use-theme/src/index.ts
@@ -46,22 +46,20 @@ export function useTheme(defaultTheme: Theme = ThemeProps.SYSTEM) {
 
   const setTheme = useCallback(
     (newTheme: Theme) => {
-      try {
-        const targetTheme =
-          newTheme === ThemeProps.SYSTEM
-            ? window.matchMedia?.(MEDIA).matches
-              ? ThemeProps.DARK
-              : ThemeProps.LIGHT
-            : newTheme;
-
-        localStorage.setItem(ThemeProps.KEY, targetTheme);
-
-        document.documentElement.classList.remove(theme);
-        document.documentElement.classList.add(targetTheme);
-        setThemeState(newTheme);
-      } catch {}
+      const targetTheme =
+        newTheme === ThemeProps.SYSTEM
+          ? window.matchMedia?.(MEDIA).matches
+            ? ThemeProps.DARK
+            : ThemeProps.LIGHT
+          : newTheme;
+
+      localStorage.setItem(ThemeProps.KEY, targetTheme);
+
+      document.documentElement.classList.remove(theme);
+      document.documentElement.classList.add(targetTheme);
+      setThemeState(newTheme);
     },
-    [theme, MEDIA],
+    [theme],
   );
 
   const handleMediaQuery = useCallback(