From 8cb7cc035684c0964cd1d7bbaae948ba44f8b431 Mon Sep 17 00:00:00 2001 From: Jared Vititoe Date: Mon, 6 Apr 2026 23:19:27 -0400 Subject: [PATCH] Fix sidebar toggle button not responding after collapse toggle-sidebar action was only in the DashboardView inline script, not in dashboard.js where toggleSidebar() is defined. Move it into the dashboard.js event delegation switch so it's guaranteed to fire. Also fix beta webhook: was using a different secret than production so Gitea pushes to development never triggered the beta deploy. Co-Authored-By: Claude Sonnet 4.6 --- assets/js/dashboard.js | 4 ++++ views/DashboardView.php | 1 - 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/assets/js/dashboard.js b/assets/js/dashboard.js index d593b3c..dd94b9e 100644 --- a/assets/js/dashboard.js +++ b/assets/js/dashboard.js @@ -226,6 +226,10 @@ document.addEventListener('DOMContentLoaded', function() { case 'open-advanced-search': if (typeof openAdvancedSearch === 'function') openAdvancedSearch(); break; + // Sidebar toggle + case 'toggle-sidebar': + toggleSidebar(); + break; // Mobile navigation case 'open-mobile-sidebar': if (typeof openMobileSidebar === 'function') openMobileSidebar(); diff --git a/views/DashboardView.php b/views/DashboardView.php index f962f81..c0a542d 100644 --- a/views/DashboardView.php +++ b/views/DashboardView.php @@ -1220,7 +1220,6 @@ document.addEventListener('click', function (e) { var target = e.target.closest('[data-action]'); if (!target) return; switch (target.getAttribute('data-action')) { - case 'toggle-sidebar': if (typeof toggleSidebar==='function') toggleSidebar(); break; case 'close-advanced-search': closeAdvancedSearch(); break; case 'reset-advanced-search': resetAdvancedSearch(); break; case 'save-filter': saveCurrentFilter(); break;