-
Notifications
You must be signed in to change notification settings - Fork 6
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 tag to button component #654
Conversation
Deploying with Cloudflare Pages
|
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #654 +/- ##
==========================================
- Coverage 22.24% 22.17% -0.07%
==========================================
Files 220 220
Lines 13081 13166 +85
Branches 236 236
==========================================
+ Hits 2910 2920 +10
- Misses 10081 10156 +75
Partials 90 90 ☔ View full report in Codecov by Sentry. |
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.
- Allow
tagProps
to be passed to toolbar component as well. If thestyle
prop is passed totagProps
, it should probably not override things like "absolute" which are in the defaultstyle
. - In button and toolbar stories, do not show example with tag in control. Show tag in specific story.
- Stories with longer tags than 1 chars. Up to 3 chars for the toolbar story to see how it looks like in that case.
@wadjih-bencheikh18 There is a conflict left to resolve |
@stropitek conflict resolved |
Closes : #649