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

fix(use-theme): remove redundant applyTheme from toggleTheme

Browse Source 
toggleTheme was calling applyTheme() inside the state updater AND
the useEffect was also applying on state change — double DOM write.
Now toggleTheme relies solely on the useEffect, matching setTheme
behavior.
This commit is contained in:
saravanakumardb1 2026-03-29 12:52:19 -07:00
parent 2688f7e3ca
commit bb86bf220e
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -80,10 +80,9 @@ export function useTheme(options?: UseThemeOptions): UseThemeReturn {
if (typeof window !== 'undefined') { if (typeof window !== 'undefined') {
window.localStorage.setItem(storageKey, next); window.localStorage.setItem(storageKey, next);
} }
applyTheme(next, attribute);
return next; return next;
}); });
}, [storageKey, attribute]); }, [storageKey]);
return { theme, setTheme, toggleTheme }; return { theme, setTheme, toggleTheme };
} }