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

add &subset=latin,latin-ext,cyrillic to Open Sans #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brozkeff
Copy link
Contributor

@brozkeff brozkeff commented Feb 2, 2023

Some languages require characters with diacritical marks. Czech language is not rendered properly. Adding latin-ext subset should fix issue for e.g. Czech language and cyrillic for Russian, Ukrainian or Bulgarian language.

More problematic are the special fonts in https://use.typekit.net/ucr7myw.css that need to have latin-ext characters as well since they don't render well for diacritical chars.

Some languages require characters with diacritical marks. Czech language is not rendered properly. Adding latin-ext subset should fix issue for e.g. Czech language and cyrillic for Russian, Ukrainian or Bulgarian language.

More problematic are the special fonts in https://use.typekit.net/ucr7myw.css that need to have latin-ext characters as well since they don't render well for diacritical chars.
@LunaUrsa
Copy link
Contributor

LunaUrsa commented Jul 3, 2023

Hey @brozkeff Thanks a ton for your pull requests, I accepted the other ones as they were just modifying text, I trust that you've tested this one, but I would like to double check, so I will loop back to this asap. I assume this fixes #15

Very much appreciate your contributions!

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

Successfully merging this pull request may close these issues.

2 participants