-
Notifications
You must be signed in to change notification settings - Fork 138
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(cards): rename aiLabel to decorator #6509
base: main
Are you sure you want to change the base?
chore(cards): rename aiLabel to decorator #6509
Conversation
✅ Deploy Preview for ibm-products-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ibm-products-web-components ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for carbon-for-ibm-products ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6509 +/- ##
==========================================
+ Coverage 79.88% 79.94% +0.05%
==========================================
Files 394 394
Lines 12877 12883 +6
Branches 4258 4264 +6
==========================================
+ Hits 10287 10299 +12
+ Misses 2590 2584 -6
|
…ilabel-to-decorator
…ub.com/sangeethababu9223/ibm-products into chore/card-rename-ailabel-to-decorator
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.
Made some suggestions to have the default decorator
styles be set without the AI styles and only add them when there’s an --ai-label
or --slug
present, similar to the implementation we see in Core.
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 add some tests for decorator?
I've added a basic test case for the decorator. |
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
Closes #6452
Rename aiLabel prop to decorator in ExpressiveCard / ProductiveCard
What did you change?
Updated the prop name
aiLabel
in Expressive Card, Productive Card and Card files todecorator
.Updated the card style to only show gradient styling when
AILabel
component is passed.How did you test and verify your work?
Storybook