bytelyst/learning_ai_common_plat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(use-theme): add SSR safety guards for window access

Browse Source 
setTheme, toggleTheme, and storage event listener now check
typeof window before accessing localStorage/addEventListener.
Prevents crashes in Next.js server components.
This commit is contained in:
saravanakumardb1 2026-03-29 12:46:18 -07:00
parent 6f4957d821
commit e194365711
1 changed files with 8 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
packages/use-theme/src/use-theme.ts
View File

@ -49,6 +49,8 @@ export function useTheme(options?: UseThemeOptions): UseThemeReturn {
// Sync across tabs via storage event
useEffect(() => {
if (typeof window === 'undefined') return;
function handleStorage(event: StorageEvent) {
if (event.key !== storageKey) {
return;
@ -65,7 +67,9 @@ export function useTheme(options?: UseThemeOptions): UseThemeReturn {
const setTheme = useCallback(
(t: Theme) => {
setThemeState(t);
window.localStorage.setItem(storageKey, t);
if (typeof window !== 'undefined') {
window.localStorage.setItem(storageKey, t);
}
},
[storageKey]
);
@ -73,7 +77,9 @@ export function useTheme(options?: UseThemeOptions): UseThemeReturn {
const toggleTheme = useCallback(() => {
setThemeState(prev => {
const next: Theme = prev === 'dark' ? 'light' : 'dark';
window.localStorage.setItem(storageKey, next);
if (typeof window !== 'undefined') {
window.localStorage.setItem(storageKey, next);
}
applyTheme(next, attribute);
return next;
});