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

chore(docs): migrate react-notifications examples to storybook #908

Merged
merged 3 commits into from
Oct 28, 2020

Conversation

hzhu
Copy link
Contributor

@hzhu hzhu commented Oct 26, 2020

Description

This PR migrates react-notifications examples to Storybook.

Checklist

  • ⬅️ renders as expected with reversed (RTL) direction
  • 🤘 renders as expected with Bedrock CSS (?bedrock)
  • ♿ analyzed via axe and evaluated using VoiceOver
  • 📝 tested in Chrome, Firefox, Safari, Edge, and IE11

@hzhu hzhu self-assigned this Oct 26, 2020
@hzhu hzhu changed the title chore(docs): migrate notifications to storybook chore(docs): migrate react-notifications examples to storybook Oct 26, 2020
@hzhu hzhu marked this pull request as ready for review October 26, 2020 20:05
@hzhu hzhu requested a review from a team as a code owner October 26, 2020 20:05
@coveralls
Copy link

coveralls commented Oct 26, 2020

Coverage Status

Coverage remained the same at 95.836% when pulling 96cc178 on hzhu/notifications-storybook into 7a8a556 on main.

@zendesk-garden zendesk-garden temporarily deployed to staging October 26, 2020 20:38 Inactive
control: 'boolean'
},
type: {
control: { type: 'select', options: ['success', 'warning', 'error', 'info'] }
Copy link
Member

Choose a reason for hiding this comment

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

Please add support for the default undefined case.

Comment on lines 23 to 29
<Title>Nori Grape Beet</Title>
<Paragraph>
Nori grape silver beet broccoli kombu beet greens fava bean potato quandong celery.
Bunya nuts black-eyed pea prairie turnip leek lentil turnip greens parsnip. Sea lettuce
lettuce water chestnut eggplant winter purslane fennel azuki bean earthnut pea sierra
leone bologi leek soko chicory celtuce parsley jícama salsify black-eyed pea quandong.
</Paragraph>
Copy link
Member

Choose a reason for hiding this comment

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

Can we see controls for the regular weight title and multi-line paragraph here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah a control could be added for that.

Comment on lines 24 to 25
Turnip greens yarrow ricebean rutabaga endive cauliflower sea lettuce kohlrabi amaranth
water
Copy link
Member

Choose a reason for hiding this comment

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

Can we see a control for a multi-line paragraph here?

@zendesk-garden zendesk-garden temporarily deployed to staging October 27, 2020 18:17 Inactive
@hzhu hzhu requested review from jzempel and a team October 27, 2020 18:19
@hzhu hzhu merged commit 1d7fa20 into main Oct 28, 2020
@hzhu hzhu deleted the hzhu/notifications-storybook branch October 28, 2020 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants