-
Notifications
You must be signed in to change notification settings - Fork 14
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: badge cleanup #221
feat: badge cleanup #221
Conversation
variant: 'orange', | ||
css: { '&:focus-visible': { boxShadow: '0 0 0 1px $colors$orange9' } }, | ||
}, | ||
{ |
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.
ℹ️ missing compoundVariant purple
</Badge> | ||
<Badge as="button" {...args} variant="orange"> | ||
Orange | ||
</Badge> |
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.
ℹ️ missing purple in story
</Badge> | ||
<Badge {...args} variant="orange"> | ||
Orange | ||
</Badge> |
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.
ℹ️ missing purple in story
@@ -69,7 +69,7 @@ | |||
"react": "17.0.1", | |||
"react-dom": "17.0.1", | |||
"rollup": "^2.33.1", | |||
"rollup-plugin-typescript2": "^0.29.0", | |||
"rollup-plugin-typescript2": "^0.30.0", |
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.
ℹ️ Check PR description
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.
LGTM
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.
LGTM
0e73a88
to
0aac6e3
Compare
🎉 This PR is included in version 1.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Cleaning up badge component, NB: this applies no design system changes.
This will be done, hopefully in a future step.
gray
) is duplicated to make sure it's clear to the userPackage changes
rollup-plugin-typescript2
from^0.29.0
to^0.30.0
due to a build issueError: '__spreadArrays' is not exported by tslib
occurs whennpm run build
alexjoverm/typescript-library-starter#310 (comment)