Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[table] fix column layout when columnWidths has sparse values #5344

Merged
merged 3 commits into from
Jun 2, 2022

Conversation

adidahiya
Copy link
Contributor

Fixes #5264

Checklist

  • Includes tests
  • Update documentation

Changes proposed in this pull request:

Fix implementation of lifecycle method in Table and Table2 components so that it computes state.columnWidths correctly even when consumers provide a props.columnWidths with sparse (undefined) values for new columns.

Reviewers should focus on:

Screenshot

2022-06-01 17 45 31

@blueprint-bot
Copy link

[table] fix column width layout with sparse values

Previews: documentation | landing | table | demo

@blueprint-bot
Copy link

revert typo renames

Previews: documentation | landing | table | demo

@adidahiya adidahiya merged commit 2a6601e into develop Jun 2, 2022
@adidahiya adidahiya deleted the ad/fix-table-column-widths branch June 2, 2022 14:36
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.

When using columnWidths prop in Table, new columns have no width
2 participants