-
Notifications
You must be signed in to change notification settings - Fork 0
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
Normalize colors and css var names for Phase 2 Redesign #2175
Merged
eddierubeiz
merged 16 commits into
phase_2_redesign_base
from
color_normalization_phase_2
Jun 6, 2023
Merged
Normalize colors and css var names for Phase 2 Redesign #2175
eddierubeiz
merged 16 commits into
phase_2_redesign_base
from
color_normalization_phase_2
Jun 6, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
And another set of mnemonic variable names
…der, some red highlights
…on't use old -dark-blue anymore
… mostly -blackish but sometimes other choices
… for links though
…lor for facet arrow
Oh, this also uses the new brand green color for bootstrap "success" theme color. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We create new SCSS variables for all applicable colors in Brand guide color pallette. We also preserve a separate set of SCSS vars with mnemonic names for the colors we actually use.
We remove all OLD brand color vars and usage, in most cases replacing them with new brand colors.
Removed a couple CSS classes that weren't actually applying to anything anymore, that we came across.
As part of Phase 2 we had a variable called
shi-dark-blue
, but after learning that Brand Guide refers to this as gray (it is a dark gray very slightly blue tinted), we changed variable name toshi-dark-gray
.We continue using a couple colors not from current Brand Guide:
Our links, we haven't yet decided what to do with them, but for now are keeping the old brand blue color, now in scss var
$shi-blue-text
.We need a gray for muted text, but none of the Brand Guide grays suffice, they are too dark or too light, skipping a middle-ground we need. Rather than preserve the exact old brand gray we were using, we take the current brand dark gray, and lighten it in color until it is very close in lightness/darkness to previous brand gray. This is now in SCSS var
$shi-alt-muted-text
.All commits: