From 3a838809c6dc735fb2e998f6955aee903b64958e Mon Sep 17 00:00:00 2001
From: "Claude (cleanup)" <claude@aegis-sight.de>
Date: Sat, 9 May 2026 14:38:36 +0000
Subject: [PATCH] fix(navigation): #healthSubTabs aus globalem Top-Tab-Handler
 ausnehmen

Der globale setupNavTabs in app.js fing nav-tab-Clicks aus ALLEN
nav-tabs ab, ausser #orgDetailTabs und #sourceSubTabs. Das neue
#healthSubTabs (aus dem letzten Commit) war nicht in der :not()-
Liste und triggerte daher den Top-Level-Handler, der getElementById("sec-suggestions")
suchte und null bekam -> Crash beim classList.add("active").

Fix: :not(#healthSubTabs) ergaenzt an allen drei Stellen
(setupNavTabs, setupNavTabs Click-Handler, openSection-Helfer in Z. 408).
Cache-Buster fuer app.js gebumpt 20260509d -> 20260509j.
---
 src/static/dashboard.html | 2 +-
 src/static/js/app.js      | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/static/dashboard.html b/src/static/dashboard.html
index 0d9c672..190f5fc 100644
--- a/src/static/dashboard.html
+++ b/src/static/dashboard.html
@@ -721,7 +721,7 @@
     </div>
   </div>
 
-  <script src="/static/js/app.js?v=20260509d"></script>
+  <script src="/static/js/app.js?v=20260509j"></script>
   <script src="/static/js/sources.js?v=20260509d"></script>
   <script src="/static/js/source-health.js?v=20260509i"></script>
   <script src="/static/js/audit.js?v=20260509d"></script>
diff --git a/src/static/js/app.js b/src/static/js/app.js
index 95dac4b..a597f3a 100644
--- a/src/static/js/app.js
+++ b/src/static/js/app.js
@@ -55,10 +55,10 @@ function logout() {
 
 // --- Navigation ---
 function setupNavTabs() {
-  document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs) .nav-tab").forEach(tab => {
+  document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs):not(#healthSubTabs) .nav-tab").forEach(tab => {
     tab.addEventListener("click", () => {
       const section = tab.dataset.section;
-      document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs) .nav-tab").forEach(t => t.classList.remove("active"));
+      document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs):not(#healthSubTabs) .nav-tab").forEach(t => t.classList.remove("active"));
       tab.classList.add("active");
       document.querySelectorAll(".app-content > .section").forEach(s => s.classList.remove("active"));
       document.getElementById(`sec-${section}`).classList.add("active");
@@ -405,7 +405,7 @@ document.addEventListener("DOMContentLoaded", () => {
 
 function switchToOrg(orgId) {
   // Switch to orgs tab and open detail
-  document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs) .nav-tab").forEach(t => t.classList.remove("active"));
+  document.querySelectorAll(".nav-tabs:not(#orgDetailTabs):not(#sourceSubTabs):not(#healthSubTabs) .nav-tab").forEach(t => t.classList.remove("active"));
   document.querySelector('.nav-tab[data-section="orgs"]').classList.add("active");
   document.querySelectorAll(".app-content > .section").forEach(s => s.classList.remove("active"));
   document.getElementById("sec-orgs").classList.add("active");