Skip to content

Commit e1e8318

Browse files
committed
fix: theme-init keeps light in css
1 parent 1473b26 commit e1e8318

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

resources/js/theme-switch-two-states/theme-init.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
if (localStorage.theme === "__DARK__") {
33
document.documentElement.classList.add("__DARK__");
44
} else if (localStorage.theme === "__LIGHT__") {
5-
document.documentElement.classList.add("__LIGHT__");
5+
// do nothing
66
} else if (window.matchMedia("(prefers-color-scheme: __DARK__)").matches) {
77
document.documentElement.classList.add("__DARK__");
88
}

tests/js/theme-switch-two-states/theme-init.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,12 @@ describe("ThemeInitScript.js IIFE behavior", () => {
7373
expect(document.documentElement.classList.contains(DEFAULT_LIGHT)).toBe(false);
7474
});
7575

76-
it("adds light class when localStorage.theme is light", () => {
76+
it("adds no class when localStorage.theme is light", () => {
7777
localStorage.setItem("theme", DEFAULT_LIGHT);
7878

7979
runThemeInitScript({ matchMediaMatches: false });
8080

81-
expect(document.documentElement.classList.contains(DEFAULT_LIGHT)).toBe(true);
81+
expect(document.documentElement.classList.contains(DEFAULT_LIGHT)).toBe(false);
8282
expect(document.documentElement.classList.contains(DEFAULT_DARK)).toBe(false);
8383
});
8484

0 commit comments

Comments
 (0)