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

Typography: add conditional to antialiased usage #5

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

danilowoz
Copy link

@danilowoz danilowoz commented Jul 26, 2023

I'd like to propose adding a conditional when using -webkit-font-smoothing: antialiased and text-rendering: optimizeLegibility for better typography legibility.

Recently, I learned that this might cause the opposite result when applied to light themes, where a dark text is on a light background. Research more about this stop, I came across this amazing article that argues that designers and front-end developers should stop using it as a fix and deliberate its usage only when is necessary, meaning stop using it as a go-for solution and only add that when fonts were not designed to full all pixes properly.

Here's an example with antialiased text in both contexts (light/dark themes). You can see that the second column on a light theme doesn't have great legibility like the non-antialiased version. While this is still subjective, I would not recommend it as a standard practice since it is not a default behavior on browsers.

image

@vercel
Copy link

vercel bot commented Jul 26, 2023

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

Name Status Preview Comments Updated (UTC)
interfaces ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 26, 2023 0:50am

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