mirror of
https://github.com/fsolla/teqo.git
synced 2026-11-13 15:13:07 +00:00
155f6492d9
The picker shipped as a capability of CampaignTable rather than a municípios feature, so every list that already uses the shared table got it: pass `columnVisibility` and the table mounts the menu and filters the columns. Persistence is a cookie, not localStorage, and the reason is a type rather than a preference: CampaignTableColumn.cell is a function, so the column array never crosses the server/client boundary and the client cannot rebuild the table without it. The server filters, so a hidden column does not reach the RSC payload either. `campaign_columns` carries a listId -> hidden ids map in a compact form with no JSON and no percent-encoding, because document.cookie and Next's cookies() disagree about decoding. Closing the menu is the commit. The first cut debounced at 400 ms, which batches a burst rather than a reading pace, so unchecking four columns while reading their labels was four full-route refreshes — on /campanha/liderancas each one re-pays the ~229 ms / 371-statement render already ledgered P1. The timer survives at 3 s as a safety net for a menu left open; the price is that the table behind the menu no longer updates per checkbox. `defaultVisible` is gone: it was true on 10 columns and false on none, and storing only the hidden set is what makes a newly shipped column visible to everyone who never touched the picker. In exchange `label` is required, which made `head` optional — 24 columns had a head that only restated the new label. Each surface declares one mandatory column, re-filtered on the server so a stale cookie cannot leave a row without its only linked cell. The bug review caught is this codebase's recurring one: the RSC payload of toggle A arrives inside the window of toggle B, and adopting the server there undid B and then committed the set without it. Reconciliation now runs only when no write is pending, pinned red against the previous code. Municipality column ids are typed against MunicipalityListColumnId now, so the id, the B22 description and the picker label are one key; the union member `nivel` was renamed `level` to match the id and the ?level= filter. The picker's own content comparison is sameIdSet, widened to string ids.
14 lines
545 B
TypeScript
14 lines
545 B
TypeScript
/**
|
|
* Order-independent equality for two id lists — the guard every list-cell
|
|
* optimistic-state hook uses to tell a genuine external prop change
|
|
* (navigation/RSC refresh) apart from a re-render carrying the same
|
|
* pre-edit content (which must not clobber in-flight optimistic state).
|
|
*/
|
|
export const sameIdSet = <Id extends string | number>(
|
|
left: readonly Id[],
|
|
right: readonly Id[],
|
|
): boolean => {
|
|
if (left.length !== right.length) return false
|
|
const rightSet = new Set(right)
|
|
return left.every((id) => rightSet.has(id))
|
|
}
|