MakeSense for GitHub — Make Sense of Any GitHub PR, with full cognitive coverage of every change.
sadmann7/tablecn#1143 — 4 files, 0 review comments
Implement Sticky Headers for Data Tables
Overview
Implement sticky headers for data tables. This adds configuration options, a styling utility, and component updates for dynamic header styling.
Technical Highlights
src/components/data-table/data-table.tsx: Updates `DataTable` to apply sticky header styles using `TableMeta` properties.
src/components/ui/table.tsx: Extends the base `Table` component with a `containerStyle` prop for its wrapper.
src/lib/data-table.ts: Adds `getStickyHeaderStyle` utility for generating sticky header CSS, including `zIndex` logic.
src/types/data-table.ts: Extends `TableMeta` with `stickyHeader` and `maxHeight` properties for configuration.
Impact
Functionality: Data tables now support sticky headers and vertical scrolling with a defined maximum height.
Risk: Verify `zIndex` interactions between sticky headers and other layered UI elements.
Files reviewed
- src/components/data-table/data-table.tsx — 0 comment(s)
- src/components/ui/table.tsx — 0 comment(s)
- src/lib/data-table.ts — 0 comment(s)
- src/types/data-table.ts — 0 comment(s)
View the full interactive review (JavaScript required).