-
Notifications
You must be signed in to change notification settings - Fork 11
feat: button new style #1121
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: button new style #1121
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1121 +/- ##
=======================================
Coverage 85.10% 85.10%
=======================================
Files 822 822
Lines 17011 17011
Branches 2158 2158
=======================================
Hits 14478 14478
Misses 2502 2502
Partials 31 31 Continue to review full report at Codecov.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@@ -3,6 +3,7 @@ export const enum ButtonRole { | |||
Primary = 'primary', | |||
Secondary = 'secondary', | |||
Tertiary = 'tertiary', | |||
Quaternary = 'quaternary', |
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.
We might want a different name for this at some point, but I can't come up with anything better right now.
@@ -123,6 +135,10 @@ | |||
@include button-style(inherit, inherit, $blue-5, $blue-4); | |||
} | |||
|
|||
&.primary-reversed { |
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.
This line needs to change also.
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.
Removed here #1123
Unit Test Results 4 files 265 suites 16m 45s ⏱️ Results for commit a99596d. |
Description
Adding new style: Primary Reversed, being similar to primary but inverting colors.
Testing
Visual Testing
Checklist:
Documentation
Make sure that you have documented corresponding changes in this repository or hypertrace docs repo if required.