OpenTechSchool / websitenext

OpenTechSchool website
https://www.opentechschool.org
8 stars 6 forks source link

Optimize Web Fonts #118

Closed xMartin closed 4 years ago

xMartin commented 4 years ago

Let's try to load only what we need in the most efficient way for better loading performance.

vercel[bot] commented 4 years ago

This pull request is being automatically deployed with ZEIT Now (learn more). To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/opentechschool/websitenext/jcc0c789h ✅ Preview: https://websitenext-git-optimize-web-fonts.opentechschool.now.sh

xMartin commented 4 years ago

@Twissi Could you check the font weights in the CSS and make sure that it matches the fonts being loaded? Roboto 300 and 500 and OpenSans 300 and 800 don't seem to be used. And OpenSans 500 is being used but not loaded.

Twissi commented 4 years ago

I see Open Sans:

Roboto:

So yes, i think we can remove Roboto 300 and 500 and OpenSans 300 and 800

So this should be fine: family=Roboto:400,500,700|Open+Sans:400,500,700

xMartin commented 4 years ago

Roboto:400,~500~,700, right?

xMartin commented 4 years ago

@Twissi We have b, strong { font-weight: bold; font-weight: 500; }. Do we want 500 for all bold in primary font? Should we add 500 to as? Why do we have bold and 500?

Twissi commented 4 years ago

~I was thinking bold === 500~ no bold is 700