-
Notifications
You must be signed in to change notification settings - Fork 63
feat(component): add responsiveness to table components plus stateless table #1719
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
base: main
Are you sure you want to change the base?
Conversation
|
41ce893 to
26e991c
Compare
|
Hello @bigcommerce/big-design-stewards this is a biggie. I'd like you to review it before removing the draft state, as it touches many parts of the tables components. I also used Sonnet to solve a few things, especially tests and such, so please advise if you find something you want to correct. I also wanna know if this should be a major update. Thanks in advance for your help! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @davelinke, can we split this StatelessTable addition in another PR. It'll be a little easier for us to review + having separate changesets.
What?
Adds responsiveness to public table components and creates a low-level stateless table option
Why?
Historically responsiveness on table has been one of our main gaps in mobile experience. This PR adds a default responsive view to tables. It also creates a low-level stateless table for applications where the table format is not aligned with the high-level components we already have
Screenshots/Screen Recordings
Testing/Proof
dev.tsxcode