-
Notifications
You must be signed in to change notification settings - Fork 26
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 component specs #821
feat(button): new component specs #821
Conversation
Deploy preview for helix-ui ready! Built with commit 368013d |
@@ -107,15 +107,15 @@ | |||
} | |||
|
|||
@mixin hxButton-active--primary { | |||
background-color: $cyan-700; | |||
background-color: $cyan-900; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
@@ -124,7 +124,7 @@ | |||
} | |||
|
|||
@mixin hxButton-hover--primary { | |||
background-color: $cyan-500; | |||
background-color: $cyan-700; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
@@ -155,23 +155,23 @@ | |||
} | |||
|
|||
@mixin hxButton-active--secondary { | |||
background-color: $cyan-700; | |||
background-color: $cyan-900; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
} | ||
|
||
@mixin hxButton-focus--secondary { | ||
box-shadow: $focus-glow; | ||
} | ||
|
||
@mixin hxButton-hover--secondary { | ||
background-color: $cyan-500; | ||
background-color: $cyan-700; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
@@ -184,7 +184,7 @@ | |||
@mixin hxButton--tertiary { | |||
background-color: transparent; | |||
border: 0; | |||
color: $cyan-900; | |||
color: $cyan-500; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
@@ -212,13 +212,13 @@ | |||
@mixin hxButton-active--tertiary { | |||
background-color: transparent; | |||
border-color: transparent; | |||
color: $cyan-700; | |||
color: $cyan-900; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
@@ -228,5 +228,5 @@ | |||
@mixin hxButton-hover--tertiary { | |||
background-color: transparent; | |||
border-color: transparent; | |||
color: $cyan-500; | |||
color: $cyan-700; |
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.
Please update to the new v2.0
color variables:
Lines 1 to 118 in eb547a4
// ============================= // | |
// ===== COLOR PALETTE 2.0 ===== // | |
// ============================= // | |
// ========================== // | |
// ===== PRIMARY COLORS ===== // | |
// ========================== // | |
// Blue - Primary | |
$blue-100: #e5f2fb; | |
$blue-300: #6ca3e1; | |
$blue-500: #367ed4; | |
$blue-700: #0051b7; | |
$blue-900: #003482; | |
// Purple - Primary | |
$purple-100: #f9e4f8; | |
$purple-300: #ad3da5; | |
$purple-500: #95098a; | |
$purple-700: #7d0572; | |
$purple-900: #64005a; | |
// Teal - Primary | |
$teal-100: #9defeb; | |
$teal-300: #34cecc; | |
$teal-500: #00bebc; | |
$teal-700: #00aab0; | |
$teal-900: #006e8d; | |
// ====================== // | |
// ===== GRAY SCALE ===== // | |
// ====================== // | |
$gray-0: #ffffff; | |
$gray-25: #fafafa; | |
$gray-50: #f5f5f5; | |
$gray-100: #eeeeee; | |
$gray-200: #e7e7e7; | |
$gray-300: #e0e0e0; | |
$gray-400: #d8d8d8; | |
$gray-500: #bdbdbd; | |
$gray-600: #9e9e9e; | |
$gray-700: #757575; | |
$gray-750: #6b6b6b; | |
$gray-800: #616161; | |
$gray-900: #424242; | |
$gray-950: #333333; | |
$gray-975: #212121; | |
$gray-1000: #000000; | |
// ========================= // | |
// ===== STATUS COLORS ===== // | |
// ========================= // | |
// Blue - Status | |
$blue-status-100: #e6eefd; | |
$blue-status-500: #296cdc; | |
$blue-status-900: #0b377f; | |
// Green - Status | |
$green-status-100: #f5fef9; | |
$green-status-500: #008b38; | |
$green-status-900: #00461c; | |
// 1.0 vars with 2.0 color palette values | |
$green-100: #f5fef9; | |
$green-500: #008b38; | |
$green-900: #00461c; | |
// Yellow - Status | |
$yellow-status-100: #ffe7b3; | |
$yellow-status-500: #ffc749; | |
$yellow-status-900: #b37c00; | |
// 1.0 vars with 2.0 color palette values | |
$yellow-100: #ffe7b3; | |
$yellow-500: #ffc749; | |
$yellow-900: #b37c00; | |
// Red - Status | |
$red-status-100: #fddcdc; | |
$red-status-500: #d6251f; | |
$red-status-900: #760300; | |
// 1.0 vars with 2.0 color palette values | |
$red-100: #fddcdc; | |
$red-500: #d6251f; | |
$red-900: #760300; | |
// ========================== // | |
// ===== ACCENT COLORS ===== // | |
// ========================== // | |
// Orange - Accent | |
$orange-accent-100: #ffe9cc; | |
$orange-accent-500: #fa9000; | |
$orange-accent-900: #ce7700; | |
// 1.0 vars with 2.0 color palette values | |
$orange-100: #ffe9cc; | |
$orange-500: #fa9000; | |
$orange-700: #dd6105; // 1.0 color var | |
$orange-900: #ce7700; | |
// Pink - Accent | |
$pink-accent-100: #fceff4; | |
$pink-accent-500: #db2b75; | |
$pink-accent-900: #960e48; | |
// Purple - Accent | |
$purple-accent-100: #f9e4f8; | |
$purple-accent-500: #95098a; | |
$purple-accent-900: #64005a; | |
// Cyan - Accent | |
$cyan-accent-100: #f6fafc; | |
$cyan-accent-500: #437d9a; | |
$cyan-accent-900: #1c3948; |
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.
Changes have been done, Please review
font-size: 0.625rem; | ||
font-weight: 400; | ||
line-height: 1.5rem; | ||
line-height: 1; |
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 should make these changes specific to the Pill component.
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.
Changes have been done, Please review
d0be969
to
768d354
Compare
b076ce4
to
5bdaa8c
Compare
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.
There is a darker blue border on the primary buttons. Please remove this border, there should be no border on primary buttons.
The secondary button should not have a fill on hover state. Should only be outlined. The color changes should be as follows:
Default: Blue-500
Hover: Blue-700
Pressed: Blue-900
src/scss/components/pill/_index.scss
Outdated
height: 1rem; | ||
line-height: 1; |
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.
Please revert these changes. I'll add the patch to PR #822.
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.
@danielmdesigns, please review commit 368013d updates to this PR #821:
https://deploy-preview-821--helix-ui.netlify.app/components/button
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.
Buttons have been corrected and look good to me. Approved.
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.
DEV LGTM
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.
Dev LGTM
Description
<hx-button>
: Implementing color palette stylesWhat are the relevant story cards/tickets? Any additional PRs or other references?
Jira: SURF-2152
Before you request a review for this PR:
yarn test
to ensure all tests passed?