Merge pull request #1545 from gethinode/radius

fix: improve init of color mode icons

Author: markdumay

assets/js/critical/color.js

@@ -43,9 +43,7 @@
       document.documentElement.setAttribute('data-bs-theme', theme)
     }
 
-    document.querySelectorAll('.navbar-mode-selector').forEach(chk => {
-      chk.checked = (document.documentElement.getAttribute('data-bs-theme') === 'light')
-    })
+    updateSelectors()
   }
 
   // alternates the currently active theme
@@ -54,6 +52,12 @@
     setTheme(target)
   }
 
+  function updateSelectors() {
+    document.querySelectorAll('.navbar-mode-selector').forEach(chk => {
+      chk.checked = (document.documentElement.getAttribute('data-bs-theme') === 'light')
+    })
+  }
+
   window.matchMedia('(prefers-color-scheme: dark)').addEventListener('change', () => {
     if (storedTheme !== 'light' || storedTheme !== 'dark') {
       setTheme(getPreferredTheme())
@@ -69,7 +73,12 @@
     })
   })
 
-  // initialize theme directly when script is invoked
+  window.addEventListener('load', () => {
+    // update the selectors when all elements are ready
+    updateSelectors()
+  })
+
+  // initialize theme as soon as possible to reduce screen flickering
   setTheme(getTheme())
 })()