-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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 display configuration options to info addon #1157
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1157 +/- ##
==========================================
+ Coverage 13.7% 13.72% +0.02%
==========================================
Files 207 207
Lines 4647 4640 -7
Branches 518 511 -7
==========================================
Hits 637 637
- Misses 3546 3557 +11
+ Partials 464 446 -18
Continue to review full report at Codecov.
|
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.
I don't think we should use context, What problem did you have that you needed context to solve?
@@ -112,8 +113,14 @@ function previewProp(val) { | |||
|
|||
export default class PropVal extends React.Component { | |||
render() { | |||
return previewProp(this.props.val); | |||
return previewProp(this.props.val, this.context); |
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 we do it without context
?
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.
@ndelangen fixed
Hey @z4o4z can you fix the merge conflicts? sorry. |
@ndelangen Hi, of course) |
Issue: #1152
What I did
Add new options:
How to test
Run
npm run storybook
to see new stories and test it!