notofonts / arabic

Noto Arabic
SIL Open Font License 1.1
15 stars 3 forks source link

U+061C (ALM) should be removed from Thaana and Nastaliq fonts #49

Closed roozbehp closed 4 years ago

roozbehp commented 8 years ago

The presence of the glyph in the Thaana font is forcing weird behavior in Android, causing the Thaana font to get selected for the colon in a sequence such as <ARABIC-INDIC DIGIT ONE, ALM, COLON>. Of course Android should know better (I'll file a separate bug for that), but the fonts should be cleaner too.

The Nastaliq font is not shipped on Android devices, but should have the glyph removed too.

PS: If there's a legitimate reason for Nastaliq and Thaana fonts to support ALM, then all fonts for scripts whose letters default to bidi=AL should have support ALM.

/cc @raphlinus

behdad commented 8 years ago

But, if it wasn't for Android's don't selection issue, we do want ALM in these fonts, right?

roozbehp commented 8 years ago

But, if it wasn't for Android's don't selection issue, we do want ALM in these fonts, right?

That's a hard question:

Theoretically, you may want a glyph similar to that of RLM and LRM so that you can do "show invisible" with it. In practice, several platforms may not know about the character ALM at all, not even know it's default-ignorable, so it may be better for the glyph to be empty and zero-advance.

So, it all depends on which platform/product and which specific version of that you are targeting.

behdad commented 8 years ago

Agree empty and zero-advance is preferable.

marekjez86 commented 4 years ago

ALM is present, but is now empty and zero width as per Google's suggestion.