-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
✅ Deploy Preview for romantic-austin-f85522 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Figtree it up
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 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: Reducing that to 0.025em looks better: But that applies across the board including the original blocks: Thoughts, @PMByrne or @matthewoden? |
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. |
Preview site is updated, ignore the status checks above. https://deploy-preview-151--romantic-austin-f85522.netlify.app/ |
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 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. |
Next v13.4.8 is released and this commit upgrades it so the dev previews work reliably. |
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:
Open Devtools
Go to the
<body>
tagFor the class
font-body
change the font-family variable from--font-figtree
to--font-gantari
or--font-noto-sans-display
Example:
Samples
Original Levenim MT
Figtree Light
Noto Sans Display Light
Gantari
Related to #137