-
Notifications
You must be signed in to change notification settings - Fork 20
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
Extend font embedding to a subset of recommended fonts? #70
Comments
Hmmm as far as I can remember, there’s been 2 major factors “preventing” us from embedding fonts until now:
So I’m definitely not against a subset of fonts we could vouch for but the 2 constraints are tricky to say the least – kerning, hinting, support, etc. can be improved dramatically on updates so we probably want to avoid updating the repo every time a font is updated. Originally posted in 58 (comment) |
As a not-ideal but good-enough solution, something like the postcss font grabber plugin should help keeping fonts relatively up to date – at least generating new builds would upload the latest version. |
Note I'm not sure it can download fonts from GitHub yet. |
And that would strip Open Type Features we have in the HTML5 patch, cf. google/fonts#1582 |
If nothing else is satisfactory.. would having a build-time script that fetches the latest versions from a CDN or something work?
We could keep this concept for the embedded fonts, as an optional module to ReadiumCSS.
Re: Open Type Features |
Indeed, and it would penalise authors who use those features as well. All I found re. PostCSS are the aforementioned font-grabber and font-magician. But those aren’t really accommodating the use-case – actually, the use case is a middle ground between those 2 plugins. And I’m not eagerly fantasising writing and maintaining a PostCSS module for that so I guess
may well be the best option. |
Note to self: add Literata and Open Dyslexic. |
Originally posted by @HadrienGardeur in #58 (comment)
The text was updated successfully, but these errors were encountered: