Merge pull request #512 from diksha78dev/fix/issue-451

Fix: Prevent ESC key listener accumulation in modal (Issue #451)

Author: devanshi14malhotra

frontend/js/library-3d.js

@@ -268,6 +268,7 @@ class BookshelfRenderer3D {
         this.isDestroyed = false;
         this._modalBackdropHandler = null;
         this._escHandler = null;
+        this._escListenerAttached = false;
 
         // Create live region for screen reader announcements
         this.liveRegion = document.createElement('div');
@@ -1379,6 +1380,7 @@ class BookshelfRenderer3D {
         if (this.modal) {
             this.modal.classList.remove('active');
             document.body.style.overflow = '';
+            this.removeEscListener();
 
             // Reset flip after transition
             setTimeout(() => {
@@ -1393,6 +1395,14 @@ class BookshelfRenderer3D {
         }
     }
 
+    removeEscListener() {
+        if (this._escHandler && this._escListenerAttached) {
+            document.removeEventListener('keydown', this._escHandler);
+            this._escListenerAttached = false;
+            this._escHandler = null;
+        }
+    }
+
     setupModalHandlers() {
         // Book flip interaction
         const bookObject = document.getElementById('book-3d-object');
@@ -1443,14 +1453,15 @@ class BookshelfRenderer3D {
             this.addManagedListener(this.modal, 'click', this._modalBackdropHandler);
         }
 
-        // ESC key to close (bind once to avoid listener leaks).
-        if (!this._escHandler) {
+        // ESC key to close - attach only once, remove on close
+        if (!this._escListenerAttached) {
             this._escHandler = (e) => {
                 if (e.key === 'Escape' && this.modal && this.modal.classList.contains('active')) {
                     this.closeModal();
                 }
             };
-            this.addManagedListener(document, 'keydown', this._escHandler);
+            document.addEventListener('keydown', this._escHandler);
+            this._escListenerAttached = true;
         }
 
         // Add to library button logic