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

YSP-353: Release Bug: Footer link style #337

Merged
merged 1 commit into from
Feb 26, 2024

Conversation

dblanken-yale
Copy link
Contributor

@dblanken-yale dblanken-yale commented Feb 22, 2024

YSP-353: Release Bug: Footer link style

Description of work

  • Adds CSS to footer links to have shadow color matching background color

Testing Link(s)

Functional Review Steps

  • Visit Component Story and verify that the links have a shadow color matching the background it's on
  • Visit multidev and verify that the footer looks right as well.

Copy link

netlify bot commented Feb 22, 2024

Deploy Preview for dev-component-library-twig ready!

Name Link
🔨 Latest commit 0ffe154
🔍 Latest deploy log https://app.netlify.com/sites/dev-component-library-twig/deploys/65dc9e81dc9dbc00087dda25
😎 Deploy Preview https://deploy-preview-337--dev-component-library-twig.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.

Copy link

@miketullo95 miketullo95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dblanken-yale dblanken-yale marked this pull request as ready for review February 22, 2024 19:31
@dblanken-yale dblanken-yale force-pushed the YSP-353-fix-footer-shadow branch from 9aa2451 to 021992a Compare February 22, 2024 19:45
Copy link
Contributor

@codechefmarc codechefmarc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. This worked for me on the Storybook instance. I double checked the dev environment and I'm not seeing this issue but did see it on the v110 environment. Glad you're going to reach out to Joe to figure out if we truly need this or not going forward.

When changing to a background that is not white, the links that are
decorated with underlines have an attribute of a white shadow on it.
This results in a terrible looking link.  The way this passed in the
past was that we had a white background with a white shadow, so it
looked ok.  This now mimics the same attempting to match the footer
background.  Right now we are just targeting the footer since it was
found only here, but it makes me wonder if we need the shadow?
@dblanken-yale dblanken-yale force-pushed the YSP-353-fix-footer-shadow branch from 021992a to 0ffe154 Compare February 26, 2024 14:21
@dblanken-yale dblanken-yale marked this pull request as draft February 26, 2024 14:48
@dblanken-yale dblanken-yale marked this pull request as ready for review February 26, 2024 14:48
@dblanken-yale dblanken-yale merged commit 82e2c89 into develop Feb 26, 2024
7 checks passed
@dblanken-yale dblanken-yale deleted the YSP-353-fix-footer-shadow branch February 26, 2024 15:06
@vinmassaro vinmassaro mentioned this pull request Feb 26, 2024
Copy link

🎉 This PR is included in version 1.39.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants