refactor(site): use .js- prefix pattern for theme toggle

eddmann · eddmann · commit 83af7e827ad8 · 2025-12-29T22:24:27.000Z
Replace inline onclick handler with addEventListener pattern
to match existing .js-copy-command convention
diff --git a/site/index.html b/site/index.html
@@ -289,8 +289,7 @@
 </head>
 <body class="min-h-screen text-slate-800 dark:text-slate-200 antialiased overflow-x-hidden bg-pattern">
     <!-- Theme Toggle Button -->
-    <button id="theme-toggle" onclick="toggleTheme()"
-        class="fixed top-6 right-6 z-50 p-3 rounded-2xl bg-white/70 dark:bg-slate-800/70 backdrop-blur-lg border border-slate-200 dark:border-slate-700 shadow-lg hover:shadow-xl transition-all duration-300"
+    <button class="js-theme-toggle fixed top-6 right-6 z-50 p-3 rounded-2xl bg-white/70 dark:bg-slate-800/70 backdrop-blur-lg border border-slate-200 dark:border-slate-700 shadow-lg hover:shadow-xl transition-all duration-300"
         aria-label="Toggle dark mode">
         <svg class="w-5 h-5 text-slate-600 dark:text-slate-300 hidden dark:block" fill="none" stroke="currentColor" viewBox="0 0 24 24">
             <path stroke-linecap="round" stroke-linejoin="round" stroke-width="2" d="M12 3v1m0 16v1m9-9h-1M4 12H3m15.364 6.364l-.707-.707M6.343 6.343l-.707-.707m12.728 0l-.707.707M6.343 17.657l-.707.707M16 12a4 4 0 11-8 0 4 4 0 018 0z"/>
@@ -555,12 +554,12 @@ <h2 class="text-3xl md:text-4xl font-bold mb-4 text-slate-900 dark:text-white">I
 
             setTheme(getThemePreference());
 
-            window.toggleTheme = function() {
+            document.querySelector('.js-theme-toggle').addEventListener('click', () => {
                 const current = document.documentElement.classList.contains('dark') ? 'dark' : 'light';
                 const next = current === 'dark' ? 'light' : 'dark';
                 localStorage.setItem(STORAGE_KEY, next);
                 setTheme(next);
-            };
+            });
 
             window.matchMedia('(prefers-color-scheme: dark)').addEventListener('change', (e) => {
                 if (!localStorage.getItem(STORAGE_KEY)) {
