]> danix's work - danix.xyz-2.git/commitdiff
fix: remove duplicate theme toggle logic from hamburger-menu.html, delegate to dedica...
authorDanilo M. <redacted>
Wed, 15 Apr 2026 13:37:29 +0000 (15:37 +0200)
committerDanilo M. <redacted>
Wed, 15 Apr 2026 13:37:29 +0000 (15:37 +0200)
themes/danix-xyz-hacker/layouts/partials/hamburger-menu.html

index 9ce3b757b3e4044bd92c426e24f59b72aa62ae41..4537fb091e1767b25cf6f6d3a23264c96be43a55 100644 (file)
@@ -54,7 +54,8 @@
     <!-- Theme toggle -->
     <div class="p-6 border-t border-border/30">
       <button
-        @click="toggleTheme(); closeMenu()"
+        id="theme-toggle"
+        @click="closeMenu()"
         class="w-full py-3 px-4 bg-surface hover:bg-surface/80 rounded flex items-center justify-center gap-2 transition-colors"
       >
         <i data-feather="moon" class="w-4 h-4"></i>
     Alpine.store('menu').close();
   }
 
-  function toggleTheme() {
-    const html = document.documentElement;
-    const isDark = html.classList.contains('theme-dark');
-    const newTheme = isDark ? 'light' : 'dark';
-    html.classList.remove('theme-light', 'theme-dark');
-    html.classList.add('theme-' + newTheme);
-    localStorage.setItem('theme', newTheme);
-    feather.replace();
-  }
-
   document.getElementById('menu-toggle').addEventListener('click', () => {
     Alpine.store('menu').toggle();
   });
 
-  document.getElementById('theme-toggle').addEventListener('click', toggleTheme);
-
   // Close menu on Escape key
   document.addEventListener('keydown', (e) => {
     if (e.key === 'Escape') {