-
Notifications
You must be signed in to change notification settings - Fork 212
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: Update CountBadge styles #2442
Conversation
bf83f96
to
942873a
Compare
1 flaky test on run #6596 ↗︎Details:
|
Test | Artifacts | |
---|---|---|
... > when the "Delete" button has focus and the tab key is pressed > should hide the popup |
Screenshots
Video
|
Review all test suite changes for PR #2442 ↗︎
display: 'inline-flex', | ||
fontFamily: system.fontFamily.default, | ||
fontSize: system.fontSize.subtext.medium, | ||
fontWeight: 700, // should use system.fontWeight.bold |
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.
do we have an issue created for fontStyles not being recognized by create Styles?
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.
use stencil now that we have it
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.
good job refactoring this Alan :)
Resolves: Workday#2397 This PR: - Rebuilds `CountBadge` to use `createComponent` and our new styling utils - Updates related tests - Updates related docs - Updates V11 upgrade guide [category:Components] Co-authored-by: manuel.carrera <manuel.carrera@workday.com>
Summary
Resolves: #2397
This PR:
CountBadge
to usecreateComponent
and our new styling utilsRelease Category
Components
Checklist
ready for review
has been added to PRFor the Reviewer
Where Should the Reviewer Start?
Areas for Feedback? (optional)
Testing Manually
Screenshots or GIFs (if applicable)
Thank You Gif (optional)