notofonts / hebrew

Noto Hebrew
SIL Open Font License 1.1
2 stars 1 forks source link

Adjust vav-vav/vav-yod/yod-yod set width in Sans/Rashi for #39 #42

Closed markhdavid closed 4 months ago

markhdavid commented 4 months ago

Sans

Layers adjusted:

Rashi

Layers adjusted:

Tested by regenerating ttf's via Glyphs App > File > Export, then running hb-view commands as per issue, but extended to test for every weight of .ttf font generated for all 3 families (Sans/Rashi/Serif). After fixes, generated .png files show good alignment in all weights of all 3 families.

markhdavid commented 4 months ago

As stated, I generated test images for every weight of all three families and verified correct output. However, I'll just present here image files corresponding to the two hb-view commands shown in issue #39, only slightly modified in that the --text-file argument is for a new file that does not contain any non-Hebrew characters other than vertical bar (|), since the generated .ttf's apparently lack all non-Hebrew characters, so they would just show up as fallback characters.

This is the modified --text-file: test-all-yiddish-ligatures-bars.txt

These are the modified commands:

hb-view --font-file NotoSansHebrew-Regular.ttf --text-file test-all-yiddish-ligatures-bars.txt --output-file "sans-ligatures.png" hb-view --font-file NotoRashiHebrew-Regular.ttf --text-file test-all-yiddish-ligatures-bars.txt --output-file "rashi-ligatures.png"

These are the resulting image files:

Sans: correct: sans-ligatures

Rashi: correct: rashi-ligatures

simoncozens commented 4 months ago

Thanks for your contribution!