Skip to content
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

Update banner color scheme #2014

Merged
merged 1 commit into from
Aug 8, 2024
Merged

Update banner color scheme #2014

merged 1 commit into from
Aug 8, 2024

Conversation

anselmbradford
Copy link
Member

@anselmbradford anselmbradford commented Aug 8, 2024

Changes

  • Update banner color scheme to change the background color to teal-dark instead of teal-mid-dark and change the visited color to teal-40.

Testing

  1. Visit banner page on PR preview https://deploy-preview-2014--cfpb-design-system.netlify.app/design-system/components/banner-notification

Screenshots

Screenshot 2024-08-08 at 1 20 27 PM

Visited link is the first link, whereas unvisted link is not visited (sorry, I should I visited the second link to align with the text).

@anselmbradford anselmbradford added the lerna-changelog/bug lerna label. DO NOT MODIFY label Aug 8, 2024
Copy link

netlify bot commented Aug 8, 2024

Thanks for the improvements! Browse a preview of your changes using the link below.

Name Link
🔨 Latest commit 440ab7e
🔍 Latest deploy log https://app.netlify.com/sites/cfpb-design-system/deploys/66b500b07ee3170009207b8e
😎 Deploy Preview https://deploy-preview-2014--cfpb-design-system.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@anselmbradford anselmbradford merged commit 88cd157 into main Aug 8, 2024
7 of 8 checks passed
@anselmbradford anselmbradford deleted the ans_archive_banner branch August 8, 2024 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lerna-changelog/bug lerna label. DO NOT MODIFY
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants