From d0f2afae5720dcd14848009be9d084922f6a71b7 Mon Sep 17 00:00:00 2001
From: Aiirondev <iron.ai.dev@gmail.com>
Date: Tue, 7 Jul 2026 22:25:55 +0200
Subject: [PATCH] slight changes to the dispaying of the Detailed Library
 Modals

---
 Web/templates/library_table.html | 74 +++++++++++++++++++++-----------
 1 file changed, 50 insertions(+), 24 deletions(-)

diff --git a/Web/templates/library_table.html b/Web/templates/library_table.html
index 43a0039..ff9706f 100644
--- a/Web/templates/library_table.html
+++ b/Web/templates/library_table.html
@@ -4,7 +4,7 @@
 
 {% block content %}
 <style>
-    /* 1. The Dark Background Overlay */
+    /* The Dark Background Overlay */
     .modal {
         position: fixed;
         top: 0;
@@ -12,13 +12,13 @@
         width: 100vw;
         height: 100vh;
         background-color: rgba(0, 0, 0, 0.6);
-        /* Ensure display is controlled by JS, but set up for centering */
+        /* Display is managed by JS (none/flex) */
         align-items: center; 
         justify-content: center;
         z-index: 1000;
     }
-    
-    /* 2. The White Modal Box */
+
+    /* The White Modal Box */
     .modal-content {
         background-color: #fff;
         padding: 20px;
@@ -26,16 +26,16 @@
         max-width: 600px;
         border-radius: 8px;
         box-shadow: 0 4px 15px rgba(0, 0, 0, 0.2);
-        
-        /* These three lines are the magic for a fixed header + scrolling body */
+
+        /* Flexbox allows the header to stay fixed while the body scrolls */
         display: flex;
         flex-direction: column;
-        max-height: 85vh; /* Prevents the box from being taller than the screen */
+        max-height: 85vh;
     }
-    
-    /* 3. The Close Button */
+
+    /* The Close Button */
     .close {
-        align-self: flex-end; /* Pushes the 'x' to the right side */
+        align-self: flex-end;
         color: #aaa;
         font-size: 28px;
         font-weight: bold;
@@ -43,19 +43,18 @@
         line-height: 1;
         margin-bottom: 10px;
     }
-    
+
     .close:hover,
     .close:focus {
         color: #000;
         text-decoration: none;
     }
-    
-    /* 4. The Scrollable Content Area */
-    #detailContent {
-        overflow-y: auto; /* Adds the scrollbar ONLY if the text is too long */
-        padding-right: 10px; /* Optional: adds a little breathing room for the scrollbar */
-    }
 
+    /* The Scrollable Content Area */
+    #detailContent {
+        overflow-y: auto; /* Adds scrollbar only if needed */
+        padding-right: 10px; /* Prevents text from rubbing against the scrollbar */
+    }
     /* Library table-only view styles */
     .library-table-container {
         max-width: 1400px;
@@ -592,14 +591,14 @@
     </div>
 </div>
 
-<!-- Detail Modal -->
+<script src="https://cdnjs.cloudflare.com/ajax/libs/dompurify/3.0.6/purify.min.js"></script>
+
 <div id="detailModal" class="modal" style="display: none;">
     <div class="modal-content">
         <span class="close" onclick="closeDetailModal()">&times;</span>
         <div id="detailContent"></div>
     </div>
 </div>
-
 <script src="https://cdn.jsdelivr.net/npm/@ericblade/quagga2/dist/quagga.js"></script>
 <script>
     // =========================================================================
@@ -1098,20 +1097,47 @@
         return div.innerHTML;
     }
 
+    // Opens the modal and fetches the data
     function showItemDetail(itemId) {
+        const detailContent = document.getElementById('detailContent');
+        const detailModal = document.getElementById('detailModal');
+
+        // 1. Show the loading state immediately
+        detailContent.innerHTML = '<p>Loading details...</p>';
+        detailModal.style.display = 'flex';
+
+        // 2. Fetch the data
         fetch(`/api/item_detail/${itemId}`)
-            .then(r => r.text())
-            .then(html => {
-                document.getElementById('detailContent').innerHTML = html;
-                document.getElementById('detailModal').style.display = 'flex';
+            .then(response => {
+                if (!response.ok) {
+                    throw new Error(`HTTP error! status: ${response.status}`);
+                }
+                return response.text();
             })
-            .catch(err => console.error('Error loading detail:', err));
+            .then(html => {
+                // 3. Clean the HTML and display it
+                detailContent.innerHTML = DOMPurify.sanitize(html);
+            })
+            .catch(err => {
+                console.error('Error loading detail:', err);
+                detailContent.innerHTML = '<p>Sorry, we could not load the item details. Please try again later.</p>';
+            });
     }
 
+    // Closes the modal via the 'x' button
     function closeDetailModal() {
         document.getElementById('detailModal').style.display = 'none';
     }
 
+    // Closes the modal if the user clicks the dark background overlay
+    window.onclick = function(event) {
+        const modal = document.getElementById('detailModal');
+        // If the click happened directly on the dark overlay (not the white box)
+        if (event.target === modal) {
+            closeDetailModal();
+        }
+    };
+
     // =========================================================================
     // 5. EVENT LISTENERS INITIALIZATION & ON-LOAD INITIALIZER
     // =========================================================================