Dashboard chart click-to-filter discards other active filters #29

New Issue

2026-08-31T21:29:39-04:00

jared commented

2026-08-31 21:29:39 -04:00

Severity: High

Follow-up to #22 (chart click-to-filter). views/DashboardView.php (~lines 294-299), gotoFilter() builds a brand-new URLSearchParams() containing only the 1-2 keys the clicked chart segment maps to, then does window.location.href = '/?' + qs. This discards every other active filter on the page (search text, saved-filter selection, date range, assignee, etc.) instead of merging into the current query string.

Repro: Apply any dashboard filter (e.g. search text "sda"), then click a priority-chart segment → lands on /?priority=2&status=... with the search term silently dropped, instead of the filtered+priority intersection the user expected.

Fix: Merge the chart segment's filter key(s) into the current URLSearchParams (parsed from location.search) instead of constructing a fresh one.

**Severity:** High Follow-up to #22 (chart click-to-filter). `views/DashboardView.php` (~lines 294-299), `gotoFilter()` builds a brand-new `URLSearchParams()` containing only the 1-2 keys the clicked chart segment maps to, then does `window.location.href = '/?' + qs`. This discards every other active filter on the page (search text, saved-filter selection, date range, assignee, etc.) instead of merging into the current query string. **Repro:** Apply any dashboard filter (e.g. search text "sda"), then click a priority-chart segment → lands on `/?priority=2&status=...` with the search term silently dropped, instead of the filtered+priority intersection the user expected. **Fix:** Merge the chart segment's filter key(s) into the *current* `URLSearchParams` (parsed from `location.search`) instead of constructing a fresh one.

jared referenced this issue

2026-08-31 21:40:10 -04:00

Advanced Search silently drops filters it doesn't represent, on submit #58

jared added the priority/high ux labels 2026-09-08 10:15:43 -04:00

jared referenced this issue from a commit

2026-09-08 11:59:38 -04:00

Chart click-to-filter now merges into the current query string (#29)

jared referenced this issue from a commit

2026-09-08 15:39:42 -04:00

Merge development into main: 15-issue triage + fix batch

Sign in to join this conversation.