From 2bda60364726ef3061eb1915ab5acdd130859109 Mon Sep 17 00:00:00 2001 From: Jared Vititoe Date: Tue, 8 Sep 2026 10:44:10 -0400 Subject: [PATCH] Chart click-to-filter now merges into the current query string (#29) gotoFilter() built a brand-new URLSearchParams containing only the clicked chart segment's filter keys, discarding every other active filter (search text, date range, saved-filter selection, etc.) on navigation. Now merges the segment's filter into the current location.search, same fix approach already applied to #22. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_01MGDKHiU5RJdo3dqQUDow3X --- views/DashboardView.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/views/DashboardView.php b/views/DashboardView.php index d210aa2..2db7bed 100644 --- a/views/DashboardView.php +++ b/views/DashboardView.php @@ -292,9 +292,10 @@ include __DIR__ . '/layout_header.php'; } function gotoFilter(params) { - var qs = new URLSearchParams(); + var qs = new URLSearchParams(window.location.search); Object.keys(params).forEach(function(k) { if (params[k] !== null && params[k] !== undefined && params[k] !== '') qs.set(k, params[k]); + else qs.delete(k); }); window.location.href = '/?' + qs.toString(); }