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

docs(DataTable): add playground story #6842

Merged

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Sep 15, 2020

This PR adds a playground story to the DataTable docs so that props like useZebraStyles, size, and stickyHeader can be tested and applied. This is useful for devs and designers when confirming positioning and alignment issues in tables

Changelog

New

  • DataTable Playground story

Changed

  • add storybook knobs to the default table stories

Testing / Reviewing

Ensure the storybook knobs for the default data table stories are correct as well as the playground story

@netlify
Copy link

netlify bot commented Sep 15, 2020

Deploy preview for carbon-elements ready!

Built with commit 1010647

https://deploy-preview-6842--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Sep 15, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 1010647

https://deploy-preview-6842--carbon-components-react.netlify.app

Copy link
Contributor

@joshblack joshblack left a comment

Choose a reason for hiding this comment

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

Ah, totally blanked on the knobs. Thanks for adding in the playground example. Left a couple of comments, namely that for most stories we want people to be able to copy and paste them without any problems. The playground story is different though and doesn't need to follow this example since it uses knobs 👍

@emyarod emyarod force-pushed the data-table-docs-playground branch from 56f098e to 85d2ab2 Compare September 16, 2020 16:23
@emyarod emyarod mentioned this pull request Sep 16, 2020
36 tasks
Copy link
Contributor

@dakahn dakahn left a comment

Choose a reason for hiding this comment

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

Seems cool to me pending @tw15egan 's changes 🏄

@emyarod emyarod force-pushed the data-table-docs-playground branch from 85d2ab2 to b19e83c Compare September 17, 2020 15:20
@tw15egan tw15egan requested a review from joshblack September 17, 2020 18:46
@kodiakhq kodiakhq bot merged commit b34d1c9 into carbon-design-system:master Sep 17, 2020
@emyarod emyarod deleted the data-table-docs-playground branch September 18, 2020 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants