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 #1

Closed
wants to merge 1 commit into from

Conversation

DestyNova
Copy link
Owner

@DestyNova DestyNova commented Feb 12, 2023

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."
@DestyNova
Copy link
Owner Author

Oops, meant to open the PR on the base repo.

@DestyNova DestyNova closed this Feb 12, 2023
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.

1 participant