TypeTogether / Playwrite

Sensei primary repository.
SIL Open Font License 1.1
104 stars 4 forks source link

Kerning #14

Closed vv-monsalve closed 6 months ago

vv-monsalve commented 1 year ago

Please give kerning an overall revision and ensure the most common cases are covered. E.g. Yo Po LY LT among others

Screen Shot 2023-08-28 at 20 15 21 Screen Shot 2023-08-28 at 19 54 14
vv-monsalve commented 10 months ago

Playwrite Austria

Screen Shot 2023-11-15 at 0 00 01
vv-monsalve commented 7 months ago

@josescaglione, the following are the main findings after reviewing the basic kerning pairs for the 20 Countries of the Batch 1 to be published.

Please review the following:

You can see all the Batch-1 fonts images here: Playwrite-Batch1-kerning-rev-cc1f569-mid-res-jpg.zip . Find below som examples

Playwrite Indonesia

PW-ID

Playwrite Mexico

PW-MX

Playwrite Vietnam

PW-VN
josescaglione commented 6 months ago

NOTE ABOUT KERNING

This font contains a very large number of alternate shapes for each letter. As a result, it was decided to build compound glyphs on the fly. The process was very successful as it maintained the amount of characters manageable but a downside of this decision is the inability of adding kerning exceptions, at least in the current stage of development.

Kerning pairs were added in such a way that they avoid clashing of certain accented characters. Example below: WhatsApp Image 2024-04-03 at 11 21 47 WhatsApp Image 2024-04-03 at 11 23 53

vv-monsalve commented 6 months ago

Closing this issue with the above note.

vv-monsalve commented 5 months ago

When the latest fonts on the lang-build at commit bf2de62 are compared with the previous ones at commit cc1f569, there are no visible kern changes.

vv-monsalve commented 4 months ago

I'll collect the reports we receive here so they can be revised on further font versions.

Playwrite PL PR

josescaglione commented 3 months ago

Kerning "Ty"

Was able to check and correct all models except those that use y.cnt and a non connecting T. In those cases the pair is unnecessary: AR BR CA CO DE_GRUND HR HR_LIJEVA SK TZ US_MOD

Screen Shot 2024-06-10 at 14 54 19
josescaglione commented 3 months ago

Kerning "@n"

added positive kerning between at and .cnt glyphs

Checked all models. This changes affects: AR AU_QLD AU_VIC BE_VLG BE_WAL BR CA CL CO CU CZ DE_LA DE_SAS ES ES_DECO FR_TRAD ID IN IT_TRAD MX NL PE PL PT RO SK TZ US_MOD US_TRAD

Screen Shot 2024-06-10 at 14 52 53 Screen Shot 2024-06-10 at 14 52 34
josescaglione commented 3 months ago

kerning pairs ".r" ".e"

Added positive kerning between period and r.ful and e.ful to avoid a collision. Checked ALL models. This affects all pairs with an entry stroke on baseline grouped in e.ful and r.ful

Screen Shot 2024-06-10 at 15 09 52
josescaglione commented 3 months ago

Fix kerning C- Glyphs with entry stroke on baseline

Added positive kerning to pairs with r.ful and e.ful and related groups. This issue affects a few models that contain print capitals in combination with full cursive lower cases. For example ES, IN and ZA

Screenshot 2024-06-12 at 07 23 39