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

Remove inappropriate generic aliases for D050000L #46

Conversation

DestyNova
Copy link
Contributor

This PR removes the generic "fantasy" font aliases for the D050000L symbol font.

The D050000L font was marked as an alias for "fantasy" but shouldn't be. As a result, if another "fantasy" font isn't installed with a higher priority, D050000L is used by some browsers (in my case Firefox, but not Chromium) when a fantasy font is requested. This results in unreadable text:

Screenshot_2023-02-12_19-04

According to the W3C CSS1 generic font-family definitions: "Fantasy fonts are primarily decorative or expressive fonts that contain decorative or expressive representations of characters. These do not include Pi or Picture fonts which do not represent actual characters."

After removing the aliases locally and running fc-cache -rv, the text is readable again (although it just uses the default sans-serif font now since I apparently don't have another fantasy font installed).

(Also see my bugrep on the Pop!_OS tracker, someone else's Fedora report and an issue in this repo that provided the solution)

The "D050000L" font was marked as an alias for "fantasy" but shouldn't be. According to the W3C CSS1 generic font-family definitions: "Fantasy fonts are primarily decorative or expressive fonts that contain decorative or expressive representations of characters. These do not include Pi or Picture fonts which do not represent actual characters."
@fabiangreffrath
Copy link
Contributor

Fixes #27

@chris-liddell chris-liddell merged commit 3c0ba3b into ArtifexSoftware:master May 3, 2023
@jendrikw
Copy link

Thanks, when can we expect a release?

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.

4 participants