-
Notifications
You must be signed in to change notification settings - Fork 24
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
Psp 8109 - color palette refactor #3951
Changes from all commits
1a45517
2fcba0f
903d5b3
297b40d
5a2c8a9
02015f2
1210ddb
a9edb92
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,22 +1,27 @@ | ||
@import './styles.scss'; | ||
|
||
.Toastify__toast.Toastify__toast--dark { | ||
color: #fff; | ||
background-color: var(--surface-color-background-dark-blue); | ||
} | ||
|
||
.Toastify__toast .Toastify__toast-body { | ||
word-break: break-word; | ||
} | ||
|
||
.Toastify__toast.Toastify__toast--dark { | ||
color: #fff; | ||
background-color: $secondary-variant-color; | ||
.Toastify__toast { | ||
word-break: break-word; | ||
filter: drop-shadow(0.2rem 0.2rem 0.4rem black); | ||
} | ||
|
||
.Toastify__toast.Toastify__toast--warning { | ||
color: $text-color; | ||
background-color: $light-accent-color; | ||
color: var(--typography-color-secondary); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. could this be done using theme instead? Something like: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No. scss files don't have access to the theme. We would need to refactor all files using scss into styled components, which is outside the scope of this palette swap story (in my opinion). |
||
background-color: var(--support-surface-color-warning); | ||
} | ||
|
||
.Toastify__toast.Toastify__toast--error { | ||
color: #fff; | ||
background-color: $danger-color; | ||
background-color: var(--surface-color-primary-danger-button-default); | ||
} | ||
|
||
.tooltip { | ||
|
@@ -69,7 +74,7 @@ | |
a { | ||
color: $link-color; | ||
&:hover { | ||
color: $secondary-variant-color; | ||
color: $link-hover-color; | ||
text-decoration: underline; | ||
} | ||
} | ||
|
@@ -92,7 +97,7 @@ a { | |
border-top-width: 0.2rem; | ||
border-bottom-width: 0.2rem; | ||
&:hover { | ||
border-color: $secondary-variant-color; | ||
border-color: $link-hover-color; | ||
} | ||
} | ||
|
||
|
@@ -101,7 +106,7 @@ a { | |
background-color: $link-color; | ||
|
||
&:hover { | ||
background-color: $secondary-variant-color; | ||
background-color: $link-hover-color; | ||
} | ||
} | ||
|
||
|
@@ -115,17 +120,17 @@ a { | |
border-top-right-radius: 0; | ||
border-bottom-right-radius: 0; | ||
&:hover { | ||
border-color: $secondary-variant-color; | ||
border-color: $link-hover-color; | ||
} | ||
} | ||
|
||
.error { | ||
color: $danger-color; | ||
color: var(--surface-color-primary-danger-button-default); | ||
} | ||
|
||
.btn.btn-success, | ||
.btn.btn-success:active { | ||
background-color: $completed-color; | ||
background-color: --support-border-color-success; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. believe you would need |
||
} | ||
|
||
ul.pagination li { | ||
|
@@ -149,10 +154,10 @@ a { | |
//by default, highlight dropdown options blue | ||
.dropdown-item, | ||
.dropdown-header { | ||
color: $text-color; | ||
color: var(--typography-color-secondary); | ||
&:hover { | ||
color: #fff; | ||
background-color: $active-color; | ||
background-color: $link-hover-color; | ||
} | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,55 +1,23 @@ | ||
$primary-color: #003366; | ||
$primary-hover-color: #335c85; | ||
$primary-light-color: #38598a; | ||
$secondary-variant-color: #428bca; | ||
$light-variant-color: #606060; | ||
$dark-variant-color: #121212; | ||
$text-color: #494949; | ||
$form-text-color: #212529; | ||
$form-control-text-color: #495057; | ||
$form-background-color: #d9d9d9; | ||
$dropdown-background-color: #32507c; | ||
$accent-color: #fcba19; | ||
$light-accent-color: #fcd573; | ||
$sres-icon-color: #172b4d; | ||
$danger-color: #d8292f; | ||
$danger-background-color: #f2dede; | ||
$light-danger-color: #fae1e7; | ||
$font-danger-color: #a12622; | ||
$icon-light-color: #3b99fc; | ||
$active-color: #007bff; | ||
$completed-color: #2e8540; | ||
$filter-background-color: #f2f2f2; | ||
$slide-out-blue: #1a5a96; | ||
$selected-color: #dff0d8; | ||
$dark-blue: #313132; | ||
$primary-background-color: white; | ||
$primary-border-color: #aaaaaa; | ||
$disabled-color: #959da5; | ||
$disabled-field-background-color: #e9ecef; | ||
$discarded-color: #bcbec5; | ||
$filter-box-color: #d9eaf7; | ||
$draft-color: #1976d2; | ||
$link-color: #1a5a96; | ||
$link-hover-color: #0056b3; | ||
$subtle-color: #aaaaaa; | ||
$summary-color: #f9f1c6; | ||
$summary-border-color: #fcba19; | ||
$font-warning-color: #6c4a00; | ||
$expired-color: #6c4a00; | ||
$expired-background-color: #f9f1c6; | ||
// table ui | ||
$table-hover-color: #dee2e6; | ||
$table-header-text-color: #494949; | ||
$table-header-bg-color: #ffffff; | ||
$table-header-border-color: #8c8c8c; | ||
$table-column-border-color: #f2f2f2; | ||
$table-even-row-bg-color: #f2f2f2; | ||
$table-odd-row-bg-color: #ffffff; | ||
// text headings - H1, H2 etc | ||
$heading-text-color: #494949; | ||
$heading-border-color: #666666; | ||
// map popup | ||
$map-popup-background-color: #fffefa; | ||
// buttons | ||
$button-info-color: #909090; | ||
@import '@/../node_modules/@bcgov/design-tokens/css/variables.css'; | ||
|
||
$pims-blue-50: #428bca; | ||
$pims-grey-80: #606060; | ||
$pims-yellow-10: #fef1d8; | ||
$pims-yellow-30: #fcba19; | ||
$pims-red-10: #f2dede; | ||
$pims-white: #f2f2f2; | ||
$pims-blue-20: #d9eaf7; | ||
$pims-yellow-100: #6c4a00; | ||
|
||
$active-action-color: $pims-blue-50; | ||
$border-outline-color: $pims-grey-80; | ||
$warning-background-color: $pims-yellow-10; | ||
$number-background-color: $pims-yellow-30; | ||
$danger-background-color: $pims-red-10; | ||
$highlight-background-color: $pims-white; | ||
$filter-box-color: $pims-blue-20; | ||
$text-warning-color: $pims-yellow-100; | ||
$header-text-color: #{'var(--theme-blue-100)'}; | ||
$header-border-color: #{'var(--theme-blue-90)'}; | ||
$link-color: #{'var(--surface-color-primary-button-default)'}; | ||
$link-hover-color: #{'var(--surface-color-primary-button-hover)'}; |
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.
nit and a question: are we attempting to map hardcoded colors like this
#fff
to our new tokens (iebc-typography-text-color
or something similar) or is that outside the scope of this and would be handled with a new tech-debt task?