From 45715d68381fac5a6f841f40c110dba31a815461 Mon Sep 17 00:00:00 2001 From: Hagernesh Date: Mon, 10 Aug 2026 12:02:19 +0000 Subject: [PATCH] fix: default DataTable status to success so rows render status was required with no default; omitting it left the body blank while the footer (which renders regardless of status) still showed "Showing 1-N of N". Rows to draw is the common case, so default to success instead of requiring every caller to pass it explicitly. Co-Authored-By: Claude Opus 5 (1M context) --- packages/ui-common/src/components/data-table/table.tsx | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/ui-common/src/components/data-table/table.tsx b/packages/ui-common/src/components/data-table/table.tsx index 957da3300..3dc2d5786 100644 --- a/packages/ui-common/src/components/data-table/table.tsx +++ b/packages/ui-common/src/components/data-table/table.tsx @@ -13,7 +13,10 @@ import { DataTableFooter } from "./footer"; export function DataTable({ columns, data, - status, + // The body only renders under "success", but the footer renders regardless — + // omitting status gave a blank table under a populated "Showing 1–N of N" + // footer. Having rows to draw is the default case, so default to success. + status = "success", onRowClick, rowStyle, rowClassName,