diff --git a/src/components/KpiCard.tsx b/src/components/KpiCard.tsx index 59e9769..afa8e9b 100644 --- a/src/components/KpiCard.tsx +++ b/src/components/KpiCard.tsx @@ -19,8 +19,17 @@ export interface KpiCardProps { label: string; value: string; /** - * The sub-pill. This is where the number gets its meaning — "1 of 84 orders" - * says something "1.2%" does not. + * Accepted and ignored, like `fill` below. + * + * This was the sub-pill — where the number got its meaning, since "1 of 84 + * orders" says something "1.2%" does not. The pill was removed from the tile + * deliberately; 87 call sites across 21 files still pass the text, so the + * prop stays declared to keep them compiling and is not read. + * + * That means every one of those 87 strings is now written and never shown. + * Either drop them from the callers or bring the pill back — right now it is + * neither, and this comment is here so that is visible rather than a puzzle + * for whoever next wonders where "84 orders" went. */ note?: string; tone?: KpiTone; @@ -39,11 +48,12 @@ export interface KpiCardProps { /** * A KPI tile. * - * Label and icon, a large tabular value, a muted sub-pill carrying the - * interpretation, and a 2px underline in the tile's tone. Sized to sit 4–6 + * Label and icon, then a large tabular value. The sub-pill and the 2px tone + * underline were both removed from the tile; `note` and `fill` still arrive + * from the call sites and are ignored — see the props above. Sized to sit 4–6 * across rather than 4 — a console is glanced at all day, not read once. */ -export function KpiCard({ label, value, note, tone = 'neutral', icon }: KpiCardProps) { +export function KpiCard({ label, value, tone = 'neutral', icon }: KpiCardProps) { const color = TONE_COLOR[tone]; return ( diff --git a/src/features/store-admin/pages/InventoryPage.tsx b/src/features/store-admin/pages/InventoryPage.tsx index e85c92f..c4addab 100644 --- a/src/features/store-admin/pages/InventoryPage.tsx +++ b/src/features/store-admin/pages/InventoryPage.tsx @@ -28,7 +28,6 @@ import { stockApi } from '@/api/stock'; import type { StockRequest } from '@/api/types'; import { KpiCard } from '@/components/KpiCard'; import { PageHeader } from '@/components/PageHeader'; -import { SectionHeader } from '@/components/SectionHeader'; import { queryKeys } from '@/queries/keys'; import { useStockRequests, useStockStatement } from '@/queries/hooks'; import { useSelection } from '@/components/useSelection'; @@ -683,7 +682,7 @@ function RequestsPanel({ {request.status === 'Pending' ? ( ) : request.status === 'Approved' ? ( - + ) : request.status === 'Received' ? ( ) : ( diff --git a/src/features/store-user/pages/StoreAccountPage.tsx b/src/features/store-user/pages/StoreAccountPage.tsx index 70dc9ef..40e4877 100644 --- a/src/features/store-user/pages/StoreAccountPage.tsx +++ b/src/features/store-user/pages/StoreAccountPage.tsx @@ -267,9 +267,13 @@ export function StoreAccountPage() { icon={} title="Shop & location" > + {/* `current` is a BRANCH, and `gettenantlocations` sends no tenant + name on it — verified on the wire, tenant 1147. So the second + half of this fallback could never fire; `gettenantinfo` is the + only call that carries the business name. */}