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(storybook): fix props not rendering for some components #7303

Merged
merged 5 commits into from
Nov 18, 2020
Merged

chore(storybook): fix props not rendering for some components #7303

merged 5 commits into from
Nov 18, 2020

Conversation

aledavila
Copy link
Contributor

@aledavila aledavila commented Nov 17, 2020

Closes #7168
Closes #7169

Changelog

Fixed:

  • Radio Button
  • Tooltip
  • Overflow Menu
  • Number Input

Changed

  • the way forward ref is being used was causing issues to prop rendering on these components
  • exporting component itself and rendering storybook using that fixed this

Testing / Reviewing

Check props show up as expected
Check that storybook isn't broken

@netlify
Copy link

netlify bot commented Nov 17, 2020

Deploy preview for carbon-elements ready!

Built with commit fea08e4

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

@netlify
Copy link

netlify bot commented Nov 17, 2020

Deploy preview for carbon-components-react ready!

Built with commit fea08e4

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

@netlify
Copy link

netlify bot commented Nov 17, 2020

Deploy preview for carbon-elements ready!

Built with commit a0aabb5

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

@tw15egan
Copy link
Collaborator

Nice work! Can we just remove these blurbs since the issue is resolved now?

Screen Shot 2020-11-17 at 1 25 22 PM

@netlify
Copy link

netlify bot commented Nov 17, 2020

Deploy preview for carbon-components-react ready!

Built with commit a0aabb5

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

@aledavila
Copy link
Contributor Author

Oh yes I hadn’t noticed those

@aledavila
Copy link
Contributor Author

@tw15egan done. Ready for review

Copy link
Collaborator

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

🙌 👍 ✅

@kodiakhq kodiakhq bot merged commit cb74016 into carbon-design-system:master Nov 18, 2020
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.

Update Storybook documentation for OverflowMenu Update Storybook documentation for Tooltip
3 participants