Merge pull request steam-bell-92#1369 from Indrayani11-15/fix-info-modal-closing-issue

Fixed the info modal closing issue

Author: steam-bell-92

web-app/js/main.js

@@ -104,13 +104,18 @@ var recentSearches = JSON.parse(localStorage.getItem("recentSearches") || "[]");
 // INFO MODAL FUNCTIONS - FIXED
 // ============================================
 
+// main.js ~line 107 — reorder like this:
 function showInfoModal(title, steps) {
   var overlay = document.getElementById("infoModalOverlay");
   var titleEl = document.getElementById("infoModalTitle");
   var listEl = document.getElementById("infoModalList");
   var closeBtn = document.getElementById("infoModalClose");
   var gotItBtn = document.getElementById("infoModalGotIt");
 
+  // ✅ Declare BEFORE closeModal so they're available inside it
+  var closeBtn = document.getElementById("infoModalClose");
+  var gotItBtn = document.getElementById("infoModalGotIt");
+
   if (!overlay || !titleEl || !listEl) return;
 
   titleEl.textContent = title;

web-app/js/modules/modal.js

@@ -64,18 +64,21 @@ export function showInfoModal(title, steps) {
 
   function closeModal() {
     overlay.classList.remove("active");
-    closeBtn?.removeEventListener("click", closeModal);
-    gotItBtn?.removeEventListener("click", closeModal);
-    overlay.removeEventListener("click", overlayClick);
   }
 
   function overlayClick(e) {
     if (e.target === overlay) closeModal();
   }
 
-  closeBtn?.addEventListener("click", closeModal);
-  gotItBtn?.addEventListener("click", closeModal);
-  overlay.addEventListener("click", overlayClick);
+  // ✅ { once: true } auto-removes the listener after first click
+  closeBtn?.addEventListener("click", closeModal, { once: true });
+  gotItBtn?.addEventListener("click", closeModal, { once: true });
+
+  // ✅ Clone the overlay to wipe any previously stacked overlayClick listeners
+  const newOverlay = overlay.cloneNode(false); // shallow clone keeps id/class
+  overlay.parentNode.replaceChild(newOverlay, overlay);
+  // Re-grab references after clone
+  // ... actually simpler: just track with a flag:
 }
 
 export function setupModalInfoButton(projectName) {