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

Variant sample grid auto-size columns and resizer #3470

Merged
merged 3 commits into from
Jan 24, 2023
Merged

Conversation

cmdcolin
Copy link
Collaborator

Ports the resizer concept from the the faceted PR and also now auto-sizes the columns using the measureGridWidth method that is used in several other data-grid usages in the codebase

@github-actions github-actions bot added the needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) label Jan 24, 2023
@cmdcolin cmdcolin added enhancement New feature or request and removed needs label triage Needs a label to show in changelog (breaking, enhancement, bug, documentation, or internal) labels Jan 24, 2023
@cmdcolin
Copy link
Collaborator Author

example screenshot

Screenshot from 2023-01-23 23-52-04

@cmdcolin cmdcolin force-pushed the variant_sample_grid branch 2 times, most recently from c3f0fad to a38ea76 Compare January 24, 2023 07:55
@codecov
Copy link

codecov bot commented Jan 24, 2023

Codecov Report

Merging #3470 (1747689) into main (a8bc2e9) will increase coverage by 0.03%.
The diff coverage is 84.61%.

@@            Coverage Diff             @@
##             main    #3470      +/-   ##
==========================================
+ Coverage   61.40%   61.43%   +0.03%     
==========================================
  Files         819      820       +1     
  Lines       29459    29477      +18     
  Branches     7047     7053       +6     
==========================================
+ Hits        18088    18108      +20     
+ Misses      11210    11208       -2     
  Partials      161      161              
Impacted Files Coverage Δ
packages/core/ui/ResizeHandle.tsx 47.36% <ø> (ø)
packages/core/ui/ResizeBar.tsx 63.63% <63.63%> (ø)
packages/core/util/index.ts 83.51% <100.00%> (+1.57%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cmdcolin cmdcolin merged commit c3274d9 into main Jan 24, 2023
@cmdcolin cmdcolin deleted the variant_sample_grid branch January 24, 2023 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant