fix: theme switcher toggling incorrectly (#553)

Timothée Andres

19.07.2025 3683a0f2b6a0750f858afd4dd3e65bb9caf3ef5b

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
const getStoredThemeStyle = () => localStorage.getItem('theme');
 
const setThemeStyle = (style) => {
  localStorage.setItem('theme', style);
  const html = document.documentElement;
  const prevTheme = [...html.classList].find((c) => c.match(/theme--(light|dark)/));
  if (prevTheme) {
    html.classList.remove(prevTheme);
  }
  html.classList.add(`theme--${style}`);
};
 
const setDarkTheme = () => {
  setThemeStyle('dark');
};
const setLightTheme = () => {
  setThemeStyle('light');
};
 
const switchTheme = () => {
  const currThemeStyle = getStoredThemeStyle();
  switch (currThemeStyle) {
    case 'light':
      setDarkTheme();
      break;
    case 'dark':
      setLightTheme();
      break;
    default:
      setLightTheme();
      break;
  }
};
 
document.addEventListener(
  'DOMContentLoaded',
  () => {
    const themeSwitcher = document.querySelector('.themeswitch');
    themeSwitcher.addEventListener('click', switchTheme, false);
  },
  false,
);
 
window
  .matchMedia('(prefers-color-scheme: dark)')
  .addEventListener('change', (e) => (e.matches ? setDarkTheme() : setLightTheme()), false);
 
const currThemeStyle = getStoredThemeStyle();
if (currThemeStyle) {
  setThemeStyle(currThemeStyle);
} else if (window.matchMedia && window.matchMedia('(prefers-color-scheme: dark)').matches) {
  setDarkTheme();
} else {
  setLightTheme();
}