diff --git a/docs/release-control/v6/internal/subsystems/storage-recovery.md b/docs/release-control/v6/internal/subsystems/storage-recovery.md index 3ac9fcc68..b3324dd09 100644 --- a/docs/release-control/v6/internal/subsystems/storage-recovery.md +++ b/docs/release-control/v6/internal/subsystems/storage-recovery.md @@ -726,6 +726,8 @@ instead of behaving like a recovery-only executive summary. That support band should stay genuinely short, usually one or two compact rows rather than four-item micro reports, so the operator can scan the summary as orientation instead of reading each card like a dense checklist. +When a second support row does not materially change triage, recovery should +prefer a single support row over filling the card just because there is room. Those support rows should also stay signal-driven. Recovery summary cards should omit zero-value or low-value support rows like `Primary Item` or `Peak Day` when they do not materially improve operator triage, so the top strip diff --git a/frontend-modern/src/components/Recovery/RecoverySummary.test.tsx b/frontend-modern/src/components/Recovery/RecoverySummary.test.tsx index a0ec800dd..b138bc18a 100644 --- a/frontend-modern/src/components/Recovery/RecoverySummary.test.tsx +++ b/frontend-modern/src/components/Recovery/RecoverySummary.test.tsx @@ -75,6 +75,7 @@ describe('RecoverySummary', () => { expect(screen.queryByText('<7d')).not.toBeInTheDocument(); expect(screen.queryByText('<1h')).not.toBeInTheDocument(); expect(screen.queryByText('<24h')).not.toBeInTheDocument(); + expect(screen.queryByText('Days Active')).not.toBeInTheDocument(); expect(screen.queryByText('Multi-platform')).not.toBeInTheDocument(); expect(screen.queryByText('Primary Platform')).not.toBeInTheDocument(); expect(screen.queryByText('Primary Item')).not.toBeInTheDocument(); diff --git a/frontend-modern/src/components/Recovery/RecoverySummary.tsx b/frontend-modern/src/components/Recovery/RecoverySummary.tsx index d8b347b4c..0a142ffc8 100644 --- a/frontend-modern/src/components/Recovery/RecoverySummary.tsx +++ b/frontend-modern/src/components/Recovery/RecoverySummary.tsx @@ -133,25 +133,16 @@ export const RecoverySummary: Component = (props) => { ); const footprintRows = createMemo(() => { - const rows: Array<{ label: string; value: string | number; valueClass?: string }> = [ + return [ { label: 'Platforms', value: platformCoverage().platformCount }, ]; - if (platformCoverage().multiPlatformCount > 0) { - rows.push({ - label: 'Multi-platform', - value: platformCoverage().multiPlatformCount, - }); - } - return rows.slice(0, 2); }); const historyRows = createMemo(() => { - const rows: Array<{ label: string; value: string | number; valueClass?: string }> = []; if (activity().latestLabel) { - rows.push({ label: 'Latest Activity', value: activity().latestLabel }); + return [{ label: 'Latest Activity', value: activity().latestLabel }]; } - rows.push({ label: 'Days Active', value: activity().activeDays }); - return rows.slice(0, 2); + return [{ label: 'Days Active', value: activity().activeDays }]; }); const MetricRows = (rowProps: {