-
Notifications
You must be signed in to change notification settings - Fork 71
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
VACMS-15308: update color variables in css (includes VACMS-14268). #19520
Conversation
…ith trimmed storybook color variables
…-update-color-variables-in-css
…css' into VACMS-15308-update-color-variables-in-css
Checking composer.lock changes... |
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.
- non blocking issue @JakeBapple:
page's active tab is no longer visible
(Left side of image is staging / right side is production)
the word "content" in Marissa's screenshot is also no longer visible as it's color has been set to white. |
It looks like the color: var(--color-absolutezero); is not defined in the scss but is being referenced. |
Checking composer.lock changes... |
Checking composer.lock changes... |
It looks like there's still a few instances of
An instance of
An instance of
A few instances of
There's one more instance of
|
.paragraphsedit-paragraph li a { backgound-image} removed due to a possibility of accessibility issues since we could not find anywhere this class is being used. |
Checking composer.lock changes... |
Checking composer.lock changes... |
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.
Great work!
Checking composer.lock changes... |
Description
Relates to #15560 and #14753
Testing done
Local test run to make sure theme and storybook compile correctly.
Screenshots
QA steps
What needs to be checked to prove this works?
Accessibility and design to check global items and spot check pages to ensure look is acceptable.
What needs to be checked to prove it didn't break any related things?
No behavior should have changed.
Definition of Done
Select Team for PR review
CMS Team
Public websites
Facilities
User support
Accelerated Publishing
Does this PR need review from a Product Owner
Needs PO review