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

Revamp core components storybook doc(pt. 5) #251

Merged

Conversation

chenesan
Copy link
Contributor

Purpose

Same as #246 but for <SplitView>, <StatusIcon>, <Switch>, <SwitchIcon>, <Tag>.

Side note:

  • Also add new autohide example for <StatusIcon>, since there's no example before.

Changes

  • /packages/storybook/examples: Update new storybook doc of <SplitView>, <StatusIcon>, <Switch>, <SwitchIcon>, <Tag>.
  • /packages/core: Update <Switch>, <SwitchIcon>, <StatusIcon> props doc for better props table description in storybook.

Risk

None.

TODOs

  • Describe what should be done outside of this PR
  • Maybe in other PRs or some manual actions.

@chenesan chenesan changed the title Feature/core docs pt5 Revamp core components storybook doc(pt. 5) Feb 10, 2020
@chenesan chenesan self-assigned this Feb 10, 2020
@chenesan chenesan added the doc label Feb 10, 2020
Copy link
Contributor

@zhusee2 zhusee2 left a comment

Choose a reason for hiding this comment

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

Thanks for the auto-hide examples!

Copy link
Contributor

@benny0642 benny0642 left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@chenesan chenesan changed the base branch from feature/core-docs-pt4 to project/update-core-components-storybook-doc February 14, 2020 07:04
Revamp core components storybook doc(pt. 6)
@chenesan chenesan merged commit c33d130 into project/update-core-components-storybook-doc Feb 14, 2020
@chenesan chenesan deleted the feature/core-docs-pt5 branch February 14, 2020 07:16
@kyoyadmoon kyoyadmoon mentioned this pull request Feb 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants