-
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(Text): add support for caption-md and caption-lg #1656
Conversation
- mark deprecated things as unioned with non-deprecated things
@@ -20,9 +20,12 @@ | |||
.text--xs { | |||
@mixin eds-theme-typography-body-xs; | |||
} | |||
.text--caption { | |||
.text--caption, .text--caption-md { |
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.
.text-caption
is preserved for compatibility at the moment. When doing a deprecation cleanup sweep, it can be removed.
Will do this one as a merge commit so that each change appears in the change log |
size-limit report 📦
|
Codecov Report
@@ Coverage Diff @@
## next #1656 +/- ##
=======================================
Coverage 93.29% 93.30%
=======================================
Files 222 222
Lines 2955 2957 +2
Branches 717 717
=======================================
+ Hits 2757 2759 +2
Misses 181 181
Partials 17 17
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@jinlee93 when you get a minute, PTAL (please take a look)? |
Summary:
Test Plan: