@shipfox/react-ui 0.15.0 → 0.17.0

package/dist/components/table/data-table.js

@@ -1,21 +1,27 @@
 import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
 import { flexRender, getCoreRowModel, getFilteredRowModel, getPaginationRowModel, getSortedRowModel, useReactTable } from '@tanstack/react-table';
+import { Card, CardContent } from '../../components/card/index.js';
 import { Checkbox } from '../../components/checkbox/index.js';
-import { Icon } from '../../components/icon/index.js';
-import { Text } from '../../components/typography/index.js';
+import { EmptyState } from '../../components/empty-state/index.js';
+import { Skeleton } from '../../components/skeleton/index.js';
 import { useEffect, useMemo, useState } from 'react';
 import { cn } from '../../utils/cn.js';
 import { Table, TableBody, TableCell, TableHead, TableHeader, TableRow } from './table.js';
 import { TablePagination } from './table-pagination.js';
-export function DataTable({ columns, data, pagination = true, pageSize = 10, pageSizeOptions, showSelectedCount = false, onRowClick, emptyState, className, ...props }) {
+export function DataTable({ columns, data, pagination = true, pageSize = 10, pageSizeOptions, showSelectedCount = false, onRowClick, emptyState, columnVisibility: controlledColumnVisibility, onColumnVisibilityChange, isLoading, className, ...props }) {
     const [sorting, setSorting] = useState([]);
     const [columnFilters, setColumnFilters] = useState([]);
-    const [columnVisibility, setColumnVisibility] = useState({});
+    const [internalColumnVisibility, setInternalColumnVisibility] = useState({});
     const [rowSelection, setRowSelection] = useState({});
     const [paginationState, setPaginationState] = useState({
         pageIndex: 0,
         pageSize
     });
+    const columnVisibility = controlledColumnVisibility ?? internalColumnVisibility;
+    const setColumnVisibility = onColumnVisibilityChange ? (visibility)=>{
+        const newVisibility = typeof visibility === 'function' ? visibility(columnVisibility) : visibility;
+        onColumnVisibilityChange(newVisibility);
+    } : setInternalColumnVisibility;
     useEffect(()=>{
         setPaginationState((prev)=>({
                 ...prev,
@@ -25,7 +31,6 @@ export function DataTable({ columns, data, pagination = true, pageSize = 10, pag
     }, [
         pageSize
     ]);
-    // Add selection column if showSelectedCount is enabled
     const columnsWithSelection = useMemo(()=>{
         if (!showSelectedCount) {
             return columns;
@@ -80,78 +85,90 @@ export function DataTable({ columns, data, pagination = true, pageSize = 10, pag
             pagination: pagination ? paginationState : undefined
         }
     });
-    return /*#__PURE__*/ _jsx("div", {
-        className: cn('rounded-8 border border-border-neutral-base overflow-hidden', className),
-        ...props,
-        children: /*#__PURE__*/ _jsxs(Table, {
-            children: [
-                table.getRowModel().rows.length > 0 ? /*#__PURE__*/ _jsx(TableHeader, {
-                    children: table.getHeaderGroups().map((headerGroup)=>/*#__PURE__*/ _jsx(TableRow, {
-                            className: "hover:bg-transparent border-b",
-                            children: headerGroup.headers.map((header)=>/*#__PURE__*/ _jsx(TableHead, {
-                                    children: header.isPlaceholder ? null : flexRender(header.column.columnDef.header, header.getContext())
-                                }, header.id))
-                        }, headerGroup.id))
-                }) : null,
-                /*#__PURE__*/ _jsx(TableBody, {
-                    children: table.getRowModel().rows?.length ? table.getRowModel().rows.map((row)=>/*#__PURE__*/ _jsx(TableRow, {
-                            onClick: ()=>onRowClick?.(row.original),
-                            "data-selected": row.getIsSelected(),
-                            className: onRowClick ? 'cursor-pointer' : '',
-                            tabIndex: onRowClick ? 0 : undefined,
-                            role: onRowClick ? 'button' : undefined,
-                            onKeyDown: (event)=>{
-                                if (!onRowClick) return;
-                                if (event.key === 'Enter' || event.key === ' ') {
-                                    event.preventDefault();
-                                    onRowClick(row.original);
-                                }
-                            },
-                            children: row.getVisibleCells().map((cell)=>/*#__PURE__*/ _jsx(TableCell, {
-                                    children: flexRender(cell.column.columnDef.cell, cell.getContext())
-                                }, cell.id))
-                        }, row.id)) : /*#__PURE__*/ _jsx(TableRow, {
-                        className: "hover:bg-transparent",
-                        children: /*#__PURE__*/ _jsx(TableCell, {
-                            colSpan: table.getAllColumns().length,
-                            className: "h-240 text-center",
-                            children: emptyState || /*#__PURE__*/ _jsxs("div", {
-                                className: "flex flex-col items-center justify-center gap-12 py-48",
-                                children: [
-                                    /*#__PURE__*/ _jsx("div", {
-                                        className: "size-32 rounded-6 bg-transparent border border-border-neutral-strong flex items-center justify-center",
-                                        children: /*#__PURE__*/ _jsx(Icon, {
-                                            name: "fileDamageLine",
-                                            className: "size-16 text-foreground-neutral-subtle",
-                                            color: "var(--foreground-neutral-subtle, #a1a1aa)"
+    const skeletonRowCount = pageSize > 5 ? 5 : pageSize;
+    if (isLoading) {
+        return /*#__PURE__*/ _jsx(Card, {
+            className: cn('p-0 gap-0', className),
+            ...props,
+            children: /*#__PURE__*/ _jsx(CardContent, {
+                className: "rounded-8 overflow-hidden p-0",
+                children: /*#__PURE__*/ _jsxs(Table, {
+                    children: [
+                        /*#__PURE__*/ _jsx(TableHeader, {
+                            children: /*#__PURE__*/ _jsx(TableRow, {
+                                className: "hover:bg-transparent border-b",
+                                children: columns.map((_, idx)=>/*#__PURE__*/ _jsx(TableHead, {
+                                        children: /*#__PURE__*/ _jsx(Skeleton, {
+                                            className: "h-16 w-24"
                                         })
-                                    }),
-                                    /*#__PURE__*/ _jsxs("div", {
-                                        className: "text-center space-y-4",
-                                        children: [
-                                            /*#__PURE__*/ _jsx(Text, {
-                                                size: "sm",
-                                                className: "text-foreground-neutral-base",
-                                                children: "No results"
-                                            }),
-                                            /*#__PURE__*/ _jsx(Text, {
-                                                size: "xs",
-                                                className: "text-foreground-neutral-muted",
-                                                children: "Looks like there are no results."
+                                    }, idx.toString()))
+                            })
+                        }),
+                        /*#__PURE__*/ _jsx(TableBody, {
+                            children: Array.from({
+                                length: skeletonRowCount
+                            }).map((_, rowIdx)=>/*#__PURE__*/ _jsx(TableRow, {
+                                    className: "hover:bg-transparent",
+                                    children: columns.map((_, colIdx)=>/*#__PURE__*/ _jsx(TableCell, {
+                                            children: /*#__PURE__*/ _jsx(Skeleton, {
+                                                className: "h-16 w-full"
                                             })
-                                        ]
-                                    })
-                                ]
+                                        }, colIdx.toString()))
+                                }, rowIdx.toString()))
+                        })
+                    ]
+                })
+            })
+        });
+    }
+    return /*#__PURE__*/ _jsx(Card, {
+        className: cn('p-0 gap-0', className),
+        ...props,
+        children: /*#__PURE__*/ _jsx(CardContent, {
+            className: "rounded-b-8 overflow-hidden p-0",
+            children: /*#__PURE__*/ _jsxs(Table, {
+                children: [
+                    table.getRowModel().rows.length > 0 ? /*#__PURE__*/ _jsx(TableHeader, {
+                        children: table.getHeaderGroups().map((headerGroup)=>/*#__PURE__*/ _jsx(TableRow, {
+                                className: "hover:bg-transparent border-b",
+                                children: headerGroup.headers.map((header)=>/*#__PURE__*/ _jsx(TableHead, {
+                                        children: header.isPlaceholder ? null : flexRender(header.column.columnDef.header, header.getContext())
+                                    }, header.id))
+                            }, headerGroup.id))
+                    }) : null,
+                    /*#__PURE__*/ _jsx(TableBody, {
+                        children: table.getRowModel().rows?.length ? table.getRowModel().rows.map((row)=>/*#__PURE__*/ _jsx(TableRow, {
+                                onClick: ()=>onRowClick?.(row.original),
+                                "data-selected": row.getIsSelected(),
+                                className: onRowClick ? 'cursor-pointer' : '',
+                                tabIndex: onRowClick ? 0 : undefined,
+                                role: onRowClick ? 'button' : undefined,
+                                onKeyDown: (event)=>{
+                                    if (!onRowClick) return;
+                                    if (event.key === 'Enter' || event.key === ' ') {
+                                        event.preventDefault();
+                                        onRowClick(row.original);
+                                    }
+                                },
+                                children: row.getVisibleCells().map((cell)=>/*#__PURE__*/ _jsx(TableCell, {
+                                        children: flexRender(cell.column.columnDef.cell, cell.getContext())
+                                    }, cell.id))
+                            }, row.id)) : /*#__PURE__*/ _jsx(TableRow, {
+                            className: "hover:bg-transparent",
+                            children: /*#__PURE__*/ _jsx(TableCell, {
+                                colSpan: table.getAllColumns().length,
+                                className: "h-240 text-center rounded-t-8",
+                                children: emptyState || /*#__PURE__*/ _jsx(EmptyState, {})
                             })
                         })
+                    }),
+                    pagination && table.getRowModel().rows?.length > 0 && /*#__PURE__*/ _jsx(TablePagination, {
+                        table: table,
+                        pageSizeOptions: pageSizeOptions,
+                        showSelectedCount: showSelectedCount
                     })
-                }),
-                pagination && table.getRowModel().rows?.length > 0 && /*#__PURE__*/ _jsx(TablePagination, {
-                    table: table,
-                    pageSizeOptions: pageSizeOptions,
-                    showSelectedCount: showSelectedCount
-                })
-            ]
+                ]
+            })
         })
     });
 }

package/dist/components/table/table.js

@@ -34,7 +34,7 @@ function TableFooter({ className, ...props }) {
 function TableRow({ className, ...props }) {
     return /*#__PURE__*/ _jsx("tr", {
         "data-slot": "table-row",
-        className: cn('group/row border-b border-border-neutral-base transition-colors', 'last:border-b-0', 'hover:bg-background-neutral-hover', 'data-[selected=true]:bg-background-neutral-pressed data-[selected=true]:hover:bg-background-neutral-pressed', className),
+        className: cn('group/row border-b border-border-neutral-base transition-colors', 'last:border-b-0 last:rounded-b-8', 'hover:bg-background-neutral-hover', 'data-[selected=true]:bg-background-neutral-pressed data-[selected=true]:hover:bg-background-neutral-pressed', className),
         ...props
     });
 }

package/dist/components/table/table.stories.components.js

@@ -1,7 +1,8 @@
 import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-runtime";
 /**
  * Shared components for Table stories
- */ import { Icon } from '../../components/icon/index.js';
+ */ import { EmptyState } from '../../components/empty-state/index.js';
+import { Icon } from '../../components/icon/index.js';
 import { useSearchContext } from '../../components/search/search-context.js';
 import { SearchEmpty, SearchFooter, SearchGroup, SearchInput, SearchItem, SearchList } from '../../components/search/search-modal.js';
 import { Text } from '../../components/typography/index.js';
@@ -10,33 +11,10 @@ import { searchJobsData } from './table.stories.data.js';
 /**
  * Empty state component for job tables
  */ export function JobsEmptyState() {
-    return /*#__PURE__*/ _jsxs("div", {
-        className: "flex flex-col items-center justify-center gap-12 py-48",
-        children: [
-            /*#__PURE__*/ _jsx("div", {
-                className: "size-32 rounded-6 bg-transparent border border-border-neutral-strong flex items-center justify-center",
-                children: /*#__PURE__*/ _jsx(Icon, {
-                    name: "shipfox",
-                    className: "size-16 text-foreground-neutral-subtle",
-                    color: "var(--foreground-neutral-subtle, #a1a1aa)"
-                })
-            }),
-            /*#__PURE__*/ _jsxs("div", {
-                className: "text-center space-y-4",
-                children: [
-                    /*#__PURE__*/ _jsx(Text, {
-                        size: "sm",
-                        className: "text-foreground-neutral-base",
-                        children: "No jobs yet"
-                    }),
-                    /*#__PURE__*/ _jsx(Text, {
-                        size: "xs",
-                        className: "text-foreground-neutral-muted",
-                        children: "Import past runs or start a runner."
-                    })
-                ]
-            })
-        ]
+    return /*#__PURE__*/ _jsx(EmptyState, {
+        icon: "shipfox",
+        title: "No jobs yet",
+        description: "Import past runs or start a runner."
     });
 }
 /**