🦋 Changeset detected

Latest commit: cabe8d9

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

✅ Deploy Preview for astro-starlight ready!

Name	Link
🔨 Latest commit	cabe8d9
🔍 Latest deploy log	https://app.netlify.com/projects/astro-starlight/deploys/6929d1dbdca1b00008070292
😎 Deploy Preview	https://deploy-preview-3549--astro-starlight.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 100 Accessibility: 100 Best Practices: 100 SEO: 100 PWA: - View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

Could you explain why these fonts are “unnecessary”?

The vendor-specific fonts (ie. -apple-system and BlinkMacSystemFont) were necessary before system-ui had broad browser support, but are now redundant.

The system-ui generic font family is now supported in all modern browsers and automatically loads the OS default font. Per MDN:

Glyphs are taken from the default user interface font on a given platform.

Removing them simplifies the font stack with zero functional impact, as system-ui handles all platforms uniformly.

The vendor-specific fonts (ie. -apple-system and BlinkMacSystemFont) were necessary before system-ui had broad browser support, but are now redundant.

That seems reasonable enough. I checked Can I Use with Starlight’s browserslist query and it does looks like system-ui covers our browser support range.

However, this PR also seems to change other stuff. The named fonts are different from the current ones. Is that intentional?

Yes, that's intentional. I replaced some of the named fonts with the "Neo-Grotesque" and "Monospace Code" stacks from modernfontstacks for a cleaner cross-platform look. Also, I removed the Emoji fonts though I couldn't find out if system-ui covers those as well so might add them back just in case.

I think it’s probably best to only remove the vendor-prefixed ones for now. That seems safe, but changing the rest of the font stack risks changing how people’s sites look which could be surprising or disruptive to them, so in the absence of a strong reason to do that, I think I’d rather stick with what we have. (We have to keep in mind that there are 10,000+ Starlight sites out there, so prefer to avoid arbitrary changes without having a clear idea of the impact.)