feat(fonts): better fallbacks

[florian-lefebvre]

Changes

• See Fonts roadmap#1039 (comment)
• Updates implementation based on capsize instead of fontaine

Testing

Should pass, manual

Docs

Changeset

[changeset-bot]

🦋 Changeset detected

Latest commit: 3f82093

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

npm i https://pkg.pr.new/astro@14713

commit: 8d571a3

[codspeed-hq]

CodSpeed Performance Report

Merging #14713 will not alter performance

_{Comparing feat/better-fonts-fallbacks (3f82093) with main (3d55c5d)}

Summary

✅ 6 untouched

[RomanHauksson]

Hey Florian, thanks so much for this fix!

I tried to test out the patch locally on astro-fonts-bug, but the built index.html had the same size adjustment + overridden metrics as when I build it with the current Astro version. I also tried to test it on academic-project-astro-template but ran into some build errors.

For both of these, I probably just made a mistake when I tried to link the locally built version of Astro. I'd investigate this further to make sure it's not an issue with your patch, but I need to put this on the backburner so I can work on some assignments due this week. I can take another look this weekend, or someone else could double check it instead.

[delucis]

@RomanHauksson Thanks for testing! If you’d like to test it, the pkg.pr.new comment above might be easier as it provides a build you can install directly with npm.

Another thing to double check might be to rm -rf .astro in the project directory in case the values are cached there.

[florian-lefebvre]

And in this case also removing node_modules/.astro. I tried locally on your repro and it seemed to work so I'm going to mark this as ready for review, but feel free to try on your side anyway!