QT/sure
1
0
Fork 0
mirror of https://github.com/we-promise/sure.git synced 2026-05-30 07:49:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(savings_goals): update ONGOING count when filtering by status or search

Browse Source 
The "ONGOING · N" badge was server-rendered with @active_goals.size and
never re-synced when the Stimulus filter hid cards. Add a count target
and update it alongside the existing empty/grid toggles.
This commit is contained in:
Guillem Arias
2026-05-11 12:50:37 +02:00
parent 638749c1d5
commit 69c45d4714
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/javascript/controllers/savings_goals_filter_controller.js
View File

@@ -5,7 +5,7 @@ import { Controller } from "@hotwired/stimulus";
// and data-goal-status; the controller toggles `.hidden` on cards
// based on the active query/chip.
export default class extends Controller {
static targets = ["input", "chip", "card", "empty", "grid"];
static targets = ["input", "chip", "card", "empty", "grid", "count"];
static values = { status: { type: String, default: "all" } };
connect() {
@@ -35,6 +35,9 @@ export default class extends Controller {
if (this.hasGridTarget) {
this.gridTarget.classList.toggle("hidden", visible === 0);
}
if (this.hasCountTarget) {
this.countTarget.textContent = visible;
}
}
selectChip(event) {

2
app/views/savings_goals/index.html.erb
View File

@@ -109,7 +109,7 @@
<div class="flex items-center gap-1.5 mb-4 text-[11px] font-medium uppercase tracking-wide text-secondary">
<span><%= t(".ongoing_section.heading") %></span>
<span class="text-subdued">·</span>
<span class="tabular-nums"><%= @active_goals.size %></span>
<span class="tabular-nums" data-savings-goals-filter-target="count"><%= @active_goals.size %></span>
</div>
<div class="grid grid-cols-1 sm:grid-cols-2 lg:grid-cols-3 gap-3.5" data-savings-goals-filter-target="grid">
<% @active_goals.each do |goal| %>