Skip to content

Improve table props and docs#79

Merged
Shrinks99 merged 6 commits into
mainfrom
table-updates
Mar 25, 2026
Merged

Improve table props and docs#79
Shrinks99 merged 6 commits into
mainfrom
table-updates

Conversation

@Shrinks99

@Shrinks99 Shrinks99 commented Mar 25, 2026

Copy link
Copy Markdown
Member

Changes

  • Shifts Table documentation to use the compositional primitives (TableContainer, TableHeader, TableRow, TableHead, TableCell) as the primary API (trying to align with other components)
  • Adds new props to table primitives: tableLayout on TableContainer, truncate on TableHead/TableCell, clickable on TableRow, and sticky on TableHeader
  • Moves clickable row styles from the high-level Table wrapper to the TableRow primitive so they work in both compositional and data-driven usage
  • Applies style prop to the outer scrollable <div> in TableContainer so devs can constrain height for sticky headers
  • Rewrites table docs with new sections for column sizing, truncation, responsive columns (container queries!), and sticky headers

Caveats

  • Sticky header border disappears while scrolling due to border-collapse limitations.
    • The preflight's scoped border-collapse: collapse conflicts with border-separate needed for sticky borders; needs a broader fix.

Shrinks99 and others added 4 commits March 24, 2026 19:01
Explicitly endorse the component-based table workflow over the data-based one for code clarity & styling ability

Co-Authored-By: Claude <claude@users.noreply.github.com>
Co-Authored-By: Claude <claude@users.noreply.github.com>
@Shrinks99 Shrinks99 requested a review from tmccoy14 March 25, 2026 06:33

@tmccoy14 tmccoy14 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@Shrinks99 Shrinks99 merged commit 1382ffd into main Mar 25, 2026
1 check passed
@Shrinks99 Shrinks99 deleted the table-updates branch March 25, 2026 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants