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
Add storybook for TopInstructions #29083
Conversation
paddingLeft: 12 | ||
} | ||
}; | ||
|
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.
yeah! Thanks for including these styles. RTL doesn't get enough love on our site
export default storybook => { | ||
const stories = storybook.storiesOf('TopInstructions', module); | ||
|
||
Object.entries(STORIES).forEach(([name, options]) => { |
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.
oh I like this set up - I don't think I've seen it before, but seems tidier than writing the stories out individually and passing different props combos
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.
Can't take credit for it - copied the setup that existed for InstructionsCSF
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.
🥇great work! this is a really useful component to have available in storybook
Creates a storybook for TopInstructions. There are still some features I would like to see added in the future but this covers the major areas.
Also move the audio button to the left side when in RTL for CSD/CSP since I saw that issue while using storybook.