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

Update some nam files and add a few new ones. #176

Merged
merged 1 commit into from
Jan 11, 2016
Merged

Update some nam files and add a few new ones. #176

merged 1 commit into from
Jan 11, 2016

Conversation

rsheeter
Copy link
Collaborator

No description provided.

@@ -187,7 +187,6 @@
0x17D9 uni17D9
0x17DA uni17DA
0x17DB uni17DB
0x17DD uni17DD
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This regresses #50

davelab6 added a commit that referenced this pull request Jan 11, 2016
Update some nam files and add a few new ones.
@davelab6 davelab6 merged commit dac01e9 into master Jan 11, 2016
@davelab6 davelab6 deleted the nam branch January 11, 2016 23:16
0xFBE4 # ARABIC LETTER E ISOLATED FORM
0xFBE5 # ARABIC LETTER E FINAL FORM
0xFBE6 # ARABIC LETTER E INITIAL FORM
0xFBE7 # ARABIC LETTER E MEDIAL FORM
0xFBE8 # ARABIC LETTER UIGHUR KAZAKH KIRGHIZ ALEF MAKSURA INITIAL FORM
0xFBE9 # ARABIC LETTER UIGHUR KAZAKH KIRGHIZ ALEF MAKSURA MEDIAL FORM
0xFBFC # ARABIC LETTER FARSI YEH ISOLATED FORM
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’d drop all Arabic presentation forms completely. If the font uses them in GSUB then pyftsubset (assuming that is what is used here) will keep them any way, if they are not used (like in Amiri) then you save file size. The only character that should be kept in this block is U+FDFA.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is fdfa for, why is it special?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

Unlike the rest of the Arabic Presentation Forms blocks, this is not really a presentation form it is a symbol of its own and can’t (or shouldn’t but people may tray) to be done as a ligature. I’m not sure how wide spread its use on the web, though, Arabic Wikipedia for example uses an image probably because most system fonts have an ugly symbol for it, not what people are used to/expecting.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Right, I wanted to drop the rest of the block, though.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I.e. drop 0xFB500xFDF2, and 0xFDFC0xFEFC.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@roozbehp says,

Contrary to what one may expect, the usage of the characters from the Arabic Presentation Form blocks is relatively common on the web. That's why we included them.

So, I'm okay leave this as is, and simply not drawing these characters in new fonts :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not really convinced they are that popular, but I won’t argue more since I was only concerned about Amiri file size and now I remembered that I strip them away in the “web” version anyway.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool - not including them is fine

vv-monsalve added a commit that referenced this pull request Jul 11, 2024
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.

3 participants