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

SUM-198 | fixup footer logo border; adjust testimonial pill; update stanford font import #70

Merged
merged 11 commits into from
Sep 23, 2024

Conversation

rebeccahongsf
Copy link
Contributor

READY FOR REVIEW

Summary

  • SUM-198 | fixup footer logo border; adjust testimonial pill; update stanford font import

Review By (Date)

  • When possible

Review Tasks

Setup tasks and/or behavior to test

  1. Review code

Associated Issues and/or People

  • SUM-198

Copy link

vercel bot commented Sep 19, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
summer ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 20, 2024 6:46pm

@@ -13,8 +13,8 @@ const PageFooter = ({...props}: Props) => {
<div className="bg-cardinal-red py-10 [&_a:hocus]:text-white [&_a]:text-white [&_a]:no-underline">
<div className="centered lg:flex lg:items-start lg:gap-20">
<div className="mb-10 text-center">
<a className="text-center font-stanford" href="https://www.stanford.edu">
<div className="text-[3.2rem]">Stanford</div>
<a className="logo text-center text-[3.4rem]" href="https://www.stanford.edu">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Util class logo from decanter contains everything we need for the Stanford branding things.

.logo {
  display: inline-block;
  font-family: Stanford, "Source Serif Pro", Georgia, Times, "Times New Roman", serif;
  font-weight: 400;
  line-height: 0.75;
  text-decoration: none;
  text-transform: none;
  transform: translateZ(0);
  font-style: normal;
  font-variant: normal;
  letter-spacing: 0;
  color: #8C1515;
  font-feature-settings: "liga" 1;
  font-variant-ligatures: discretionary-ligatures;
  -webkit-font-smoothing: antialiased;
  -moz-osx-font-smoothing: grayscale;
}
.logo:focus {
  color: #8C1515;
}
.logo:hover {
  color: #8C1515;
}

We only need to add stanford: ['var(--font-stanford)', 'sans-serif'] to the Tailwind config if we want to use font-stanford elsewhere.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'm good with that. But now we need to change the font size on the line "University" in the footer. Also the hover color needs changed.

@pookmish pookmish merged commit b1d93d8 into 1.x Sep 23, 2024
4 checks passed
@pookmish pookmish deleted the bug/SUM-198--qa branch September 23, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants