fix: ignore DOMException by accessing localStorage (#123)

* fix: ignore DOMException by accessing localStorage * chore: use as Theme instead of ts-expect-error
koba04 · Mar 23, 2023 · bf93b8c · bf93b8c · vercel · Mar 23, 2023
1 parent fccbd09
commit bf93b8c
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/packages/swr-devtools-panel/src/components/ThemeProvider.tsx b/packages/swr-devtools-panel/src/components/ThemeProvider.tsx
@@ -5,8 +5,13 @@ const THEME_KEY = "$swr-devtools:theme";
 
 const initialTheme = (): Theme => {
   if (typeof window === "undefined") return "system";
-  // @ts-expect-error
-  return localStorage.getItem(THEME_KEY) || "system";
+  let theme: Theme = "system";
+  try {
+    theme = localStorage.getItem(THEME_KEY) as Theme;
+  } catch {
+    // noop
+  }
+  return theme;
 };
 
 const getThemeByPreference = () =>
@@ -42,7 +47,11 @@ export const ThemeProvider = ({ children }: { children: React.ReactNode }) => {
     setTheme(initialTheme());
   }, []);
   useEffect(() => {
-    localStorage.setItem(THEME_KEY, theme);
+    try {
+      localStorage.setItem(THEME_KEY, theme);
+    } catch {
+      // noop
+    }
   }, [theme]);
   return (
     <ThemeContext.Provider value={{ theme, setTheme }}>