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

Replace Levenim MT with Figtree Light #151

Merged
merged 13 commits into from Jul 14, 2023
Merged

Replace Levenim MT with Figtree Light #151

merged 13 commits into from Jul 14, 2023

Conversation

nerddtvg
Copy link
Contributor

@nerddtvg nerddtvg commented Apr 26, 2023

Edit June 20: Removed testing fonts, left only Figtree.

Problem: The Levenim MT font is a licensed font ($50/250k views). We need to find an alternative in the next few months before our temporary license runs out.

This PR is to enable a publicly testable website that includes several fonts for testing:

I think Figtree looks the best and is relatively close to the original font. Plus, the anti-aliasing provided from it looks much nicer on scaled down text like the menus and smaller paragraphs. My next suggestion would be Noto Sans Display. And for something styled a bit different but still sans I would choose Gantari.

To test:

  1. Figtree is default
  2. To change:
    1. Open Devtools

    2. Go to the <body> tag

    3. For the class font-body change the font-family variable from --font-figtree to --font-gantari or --font-noto-sans-display

      Example:
      image

Samples

Original Levenim MT
image

Figtree Light
image

Noto Sans Display Light
image

Gantari
image

Related to #137

@nerddtvg nerddtvg requested a review from a team as a code owner April 26, 2023 13:05
@netlify
Copy link

netlify bot commented Apr 26, 2023

Deploy Preview for romantic-austin-f85522 ready!

Name Link
🔨 Latest commit c9c06cc
🔍 Latest deploy log https://app.netlify.com/sites/romantic-austin-f85522/deploys/64a3369293200b00089569cf
😎 Deploy Preview https://deploy-preview-151--romantic-austin-f85522.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

@matthewoden matthewoden left a comment

Choose a reason for hiding this comment

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

Figtree it up

@nerddtvg nerddtvg changed the title Testing Fonts Replace Levenim MT with Figtree Light Jun 20, 2023
@nerddtvg
Copy link
Contributor Author

I found some pages where the waves go over the header text, however it does that on the current live site as well and is still legible:

https://deploy-preview-151--romantic-austin-f85522.netlify.app/services/full-stack-development
https://deploy-preview-151--romantic-austin-f85522.netlify.app/case-studies/fortune-500-insurance-client
https://deploy-preview-151--romantic-austin-f85522.netlify.app/case-studies/fortune-500-energy-client
https://deploy-preview-151--romantic-austin-f85522.netlify.app/case-studies/fortune-500-healthcare-client

Otherwise I went through the main pages plus 9 blog posts and the font seems fine. I think in the larger blocks of text on the careers, case study, and blog pages the letter spacing is too much and seems too wide:

image

Reducing that to 0.025em looks better:

image

But that applies across the board including the original blocks:

No spacing:
image

0.05em spacing:
image

0.025em spacing (suggested):
image

Thoughts, @PMByrne or @matthewoden?

@nerddtvg
Copy link
Contributor Author

I have merged the latest main and reduced the sample size to 0.025em to refresh the preview. There is simply too much spacing between letters in paragraphs with 0.05em, see my sample below.

0.05em:
image

0.025em:
image

@nerddtvg
Copy link
Contributor Author

Netlify build errors are related to:

This was introduced with next v13.2.4 and we are on v13.3.0. The fix was merged into the canary branch yesterday. It will eventually be released in v13.4.8 in a few days. Until then, we just have to keep spamming the retry button in Netlify.

@nerddtvg
Copy link
Contributor Author

Preview site is updated, ignore the status checks above.

https://deploy-preview-151--romantic-austin-f85522.netlify.app/

@nerddtvg
Copy link
Contributor Author

It seems we have an uneasy consensus. Almost everyone has said the font is a bit too light and I agree. We may be able to adjust the body/header weights down the road. The regular weight font was too heavy for the entire site as it is currently setup, but if we split out the headers to use the heading style we can adjust them independently and get a better balance.

So far, this is the closest I could find to the original font given but of course nothing is perfect. I'd like to move ahead with this for now and we can adjust the more specific styling once the base change is applied.

@nerddtvg
Copy link
Contributor Author

nerddtvg commented Jul 3, 2023

Next v13.4.8 is released and this commit upgrades it so the dev previews work reliably.

@PMByrne PMByrne merged commit 04b64bd into main Jul 14, 2023
5 checks passed
@nerddtvg nerddtvg deleted the test-font-quicksand branch October 10, 2023 15:34
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.

None yet

3 participants