-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: introduce badges addon for storybook #895
Conversation
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.
size-limit report
Path | Size |
---|---|
components | 72.4 KB (0%) |
styles | 4.47 KB (0%) |
}, | ||
beta: { | ||
...defaultBadgesConfig[BADGE.BETA], | ||
tooltip: { |
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.
nit: could we remove the tooltips for now, until we find a use for them? 🙂
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'm down for removing the tooltips, mostly to show configuration possibility
|
||
const Example = () => <p>Example to show badges addon</p>; | ||
|
||
export default { |
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.
Is this mostly for internal documentation? Would suggest naming it title: "Documentation/BadgesAddon"
(or once we merge with next
, adding to one of the comprehensive docs files there)
oh derp just saw your comment -- removing before merge also works
Codecov Report
@@ Coverage Diff @@
## main #895 +/- ##
=======================================
Coverage 95.85% 95.85%
=======================================
Files 143 143
Lines 1134 1134
Branches 171 171
=======================================
Hits 1087 1087
Misses 46 46
Partials 1 1 Continue to review full report at Codecov.
|
@jinlee93 This is awesome, thanks! When you feel good about this (looks like there's a little cleanup), would you mind also getting this setup into |
* feat: introduce badges addon for storybook * chore: remove badge tooltip config * chore: remove temp example story
Summary:
deprecated
orbeta
, wouldn't need badge otherwise?Test Plan: