feat(ui): roll slim click-to-filter stat strip out to all registers#158
Merged
Conversation
…156) Adopts the shared StatStrip (from #157) across Suppliers, Assets, Legal, Systems, Incidents, Corrective Actions, Objectives, Changes and Risks — replacing the tall stat-card grids that pushed each list far down. Counts + filtering stay glanceable in a fraction of the height. StatStrip gains an optional per-chip `static` flag for derived metrics that aren't a filter dimension (Critical / Severe / High), rendered as display-only chips. Risk's summary chips are display-only (they span different filter dimensions) and its Risk Map stays in its collapsible "More". Corrective Actions' summary cards become click-to-filter for the first time, matching the other registers. Closes #156.
unidoc-alip
approved these changes
Jul 10, 2026
unidoc-alip
left a comment
unidoc-alip
left a comment
Collaborator
There was a problem hiding this comment.
Clean, mechanical rollout of the shared StatStrip component (from #157) across all nine remaining registers, closing #156. StatStrip gained a well-scoped static flag for display-only chips (Critical/Severe/High), and every view's statusStats/summaryStats computed correctly mirrors the pre-existing stats fields, filter keys, and colors from the card grid it replaces — no dropped states, no orphaned filterStatus wiring, no v-html/new input surfaces. Verified npm run build is green and traced each converted view's fallback logic (stats.total || total || 0, etc.) against the prior markup; all matches.
No must-fix or should-fix issues found.
LGTM
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rolls the slim, always-visible StatStrip (introduced for Tasks in #157) out to the remaining registers, closing #156.
Each register opened with a grid of big stat cards that pushed the list well below the fold. This swaps them for the shared chip strip: count + label, click-to-filter, active chip highlighted — same data, same filtering, a fraction of the height.
Converted: Suppliers, Assets, Legal, Systems, Incidents, Corrective Actions, Objectives, Changes, Risks.
Notes
StatStripgains an optional per-chipstaticflag — display-only chips for derived metrics that aren't a filter dimension (Critical / Severe / High).v-modelstrip. Can follow up separately if wanted.just build-webgreen. No API/logic changes beyond wiring the existing filter refs to the strip.