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.

Sign up for GitHub

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

Jump to bottom

Update fontinfo.plist #49

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

Update fontinfo.plist #49

wants to merge 1 commit into from

Conversation

DIV-on-github
Copy link

Attempting to address issues as

@DIV-on-github
Update fontinfo.plist
7a65b2c 
Attempting to address issues as
* missing Vendor ID linked to embeddability issues (and missing licence information) mozilla#48
* missing PANOSE data mozilla#28
@DIV-on-github
Copy link
Author

DIV-on-github commented Jun 8, 2023 •
edited
Loading

Please double-check the proposed PANOSE data.
In hindsight I was probably too hasty to classify the font's proportion as "modern". It is supposed to be classified strictly by calculation.
A couple of other PANOSE parameters I left as 0 ("any"), but these too could be classified more precisely with a bit of effort from those in the know.
Even so, on balance I think it's better to —hypothetically — have several correct parameters specified, and a couple nearly correctly specified, rather than leave all parameters blank.

Once the changes are checked (and amended as needed), they should be replicated in all other fonts of this family. (The PANOSE weights of course differ between fonts, but — strictly speaking — so too may e.g. the proportion.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@DIV-on-github