Merge pull request steam-bell-92#1169 from anujsharma8d/fix/multiple-toggletheme-button

steam-bell-92 · web-flow · commit a6e5f665a88f · 2026-06-15T22:15:05.000+05:30
fix: multipe theme toggle buttons
diff --git a/web-app/index.html b/web-app/index.html
@@ -34,7 +34,7 @@
     />
   </head>
   <body class="sidebar-active">
-    <button aria-label="Switch theme" class="theme-toggle fixed-theme-toggle" type="button">
+    <button aria-label="Switch theme" class="theme-toggle fixed-theme-toggle" id="fixed-theme-toggle" type="button">
       <i class="fas fa-moon" aria-hidden="true"></i>
     </button>
     <!-- Scroll Progress Bar -->
diff --git a/web-app/js/main.js b/web-app/js/main.js
@@ -728,6 +728,17 @@ document.addEventListener("DOMContentLoaded", function () {
 
       document.body.classList.toggle("sidebar-active", showSidebar);
       console.log('Sidebar active:', showSidebar, 'scrollY:', window.scrollY);
+
+      // Hide fixed-theme-toggle if sidebar is active
+
+      const fixedThemeToggle = document.getElementById("fixed-theme-toggle");
+      if(showSidebar){
+        fixedThemeToggle.style.display = "none";
+      }
+      else{
+        fixedThemeToggle.style.display = "block";
+      }
+
     };
 
     window.addEventListener('scroll', checkAndToggleSidebar);
@@ -1293,7 +1304,7 @@ document.addEventListener("DOMContentLoaded", function () {
       }, 50);
     }
     lastFocusedElement = null;
-  })
+  }
 
   if (modalClose) modalClose.addEventListener("click", closeProjectSafe);
   if (modal) {
@@ -1851,5 +1862,4 @@ projectCards.forEach(card => {
   // Initial card filtering state update
   updateProjectVisibility(currentCategory, currentSearchQuery);
   window.updateRecentlyViewed();
-}
 });
