From c0c38268f53e6b1a7a02ad02f56ddbea1514b69f Mon Sep 17 00:00:00 2001
From: Broque Thomas <26755000+Nezreka@users.noreply.github.com>
Date: Sat, 18 Apr 2026 21:49:30 -0700
Subject: [PATCH] Fix tool help modal not closable from Automations page

The close button and backdrop click handlers were only attached when
the Tools page was visited (initializeToolHelpButtons). Automation
builder '?' buttons open the same modal but the close handlers were
never set up. Added inline onclick handlers to the modal HTML and a
global Escape key listener so closing works from any page.
---
 webui/index.html       | 4 ++--
 webui/static/script.js | 9 ++++++++-
 2 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/webui/index.html b/webui/index.html
index 1910d3a9..29442291 100644
--- a/webui/index.html
+++ b/webui/index.html
@@ -7705,11 +7705,11 @@
         </div>
     </div>
 
-    <div class="tool-help-modal" id="tool-help-modal">
+    <div class="tool-help-modal" id="tool-help-modal" onclick="if(event.target===this)closeToolHelpModal()">
         <div class="tool-help-modal-content">
             <div class="tool-help-modal-header">
                 <h3 id="tool-help-modal-title">Tool Information</h3>
-                <button class="tool-help-modal-close">&times;</button>
+                <button class="tool-help-modal-close" onclick="closeToolHelpModal()">&times;</button>
             </div>
             <div class="tool-help-modal-body" id="tool-help-modal-body">
                 <!-- Content will be dynamically inserted -->
diff --git a/webui/static/script.js b/webui/static/script.js
index c0d1038e..a20d8c69 100644
--- a/webui/static/script.js
+++ b/webui/static/script.js
@@ -24638,9 +24638,16 @@ function openToolHelpModal(toolId) {
 
 function closeToolHelpModal() {
     const modal = document.getElementById('tool-help-modal');
-    modal.classList.remove('active');
+    if (modal) modal.classList.remove('active');
     document.body.style.overflow = ''; // Restore scrolling
 }
+// Global Escape key handler for tool help modal (works even if Tools page wasn't visited)
+document.addEventListener('keydown', (e) => {
+    if (e.key === 'Escape') {
+        const modal = document.getElementById('tool-help-modal');
+        if (modal && modal.classList.contains('active')) closeToolHelpModal();
+    }
+});
 
 // ===============================
 // == RETAG TOOL FUNCTIONS      ==