fix(components): DataTable has layout prop for fixed or auto#2883
Merged
taylorvnoj merged 8 commits intomasterfrom Feb 12, 2026
Merged
fix(components): DataTable has layout prop for fixed or auto#2883taylorvnoj merged 8 commits intomasterfrom
taylorvnoj merged 8 commits intomasterfrom
Conversation
Deploying atlantis with
|
| Latest commit: |
d962d78
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://9540cfe1.atlantis.pages.dev |
| Branch Preview URL: | https://taylor-datatable-fixed.atlantis.pages.dev |
taylorvnoj
force-pushed
the
TAYLOR/datatable-fixed
branch
from
bae19ca to
afaa309
Compare
taylorvnoj
commented
Feb 9, 2026
Contributor
Author
There was a problem hiding this comment.
The difference here is the equal column sizing we're now getting from using fixed.
|
Published Pre-release for 913588d with versions: To install the new version(s) for Web run: |
| </Cluster> | ||
| </Stack> | ||
| ), | ||
| }, |
Contributor
There was a problem hiding this comment.
Mobile content is duplicated. It causes weird rendering
Comment on lines
+78
to
+79
| <DataTable.HeaderCell>Description</DataTable.HeaderCell>{" "} | ||
| {/* Takes remaining space */} |
Contributor
There was a problem hiding this comment.
Suggested change
| <DataTable.HeaderCell>Description</DataTable.HeaderCell>{" "} | |
| {/* Takes remaining space */} | |
| {/* Takes remaining space */} | |
| <DataTable.HeaderCell>Description</DataTable.HeaderCell> |
Minor change to make it clear which cell is taking the remaining space
Contributor
Author
There was a problem hiding this comment.
Yup that makes more sense, thank you! Fixed here 3e20a50
MichaelParadis
requested changes
Feb 10, 2026
Contributor
MichaelParadis
left a comment
MichaelParadis
left a comment
There was a problem hiding this comment.
Minor suggestions. Main one is the storybook rendering. I am fine either way with the docs ordering but I found it was more clear to me
taylorvnoj
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivations
Previously,
table-layout: autosized columns from content, making it hard to create consistent layouts. Switching totable-layout: fixedlets consumers control column widths (e.g. via percentages), with more predictable layouts across different data sets.To keep flexibility, we expose an optional
layoutprop so tables can use eitherfixedorauto(default) layout as needed.Changes
Changed
DataTableTable.tsx"auto" | "fixed")autoDataTableNotes.mdxDataTable.HeaderCellvia the style propComposable.stories.tsxTesting
Go to ColumnWidths Story http://localhost:6005/?path=/story/components-lists-and-tables-datatable-web-composable--column-widths
You can go to the Basic example and apply the
layoutprop, seeing the slight differences betweenautoandfixedChanges can be
tested via Pre-release
In Atlantis we use Github's built in pull request reviews.