Skip to content

Conversation

Rich-Harris
Copy link
Member

No description provided.

Copy link

vercel bot commented Oct 1, 2024

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

Name Status Preview Comments Updated (UTC)
omnisite ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 2, 2024 9:07pm

@Rich-Harris
Copy link
Member Author

I think this is way less nice than the garamond

@dummdidumm
Copy link
Member

This is much more readable/legible for me compared to garamond:

  • text is spaced out better on the y-axis, garmond feels to crammed for me
  • serif comes through better, the edges where it gets thinner are still sharp, garmond felt too thin and pixelated around the edges
  • e is way nicer

@benmccann
Copy link
Member

benmccann commented Oct 2, 2024

This one looks very nice to me too. It's my new favorite out of the options even over both the larger Garamond on main and the Tiempos PR

@benmccann
Copy link
Member

Wow. Better and cheaper!

Screenshot from 2024-10-02 13-19-52

@Rich-Harris
Copy link
Member Author

Added Freight Display (for headings) and Freight Sans (for UI). Not sure if I'm sold on the sans. It feels very similar to Adobe Clean, and the last thing I want to think when I'm on the site is that I'm in the middle of a fucking Adobe app.

@benmccann
Copy link
Member

I don't know. Now I'm preferring main to both Tiempos and this PR. I might have to take another look later on with fresh eyes

@dummdidumm
Copy link
Member

Kinda crazy, but I have the same feeling. The es are still a gripe, but other than that Garmond is growing on me

@Rich-Harris Rich-Harris mentioned this pull request Oct 2, 2024
@Rich-Harris Rich-Harris closed this Oct 3, 2024
@dummdidumm dummdidumm deleted the freight-text-pro branch October 3, 2024 05:55
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.

3 participants