Clusters with multiple spacing marks have extra advance width

dscorbett commented 3 months ago

Font

NotoSansLimbu-Regular.otf

Where the font came from, and when

Site: https://github.com/notofonts/limbu/releases/tag/NotoSansLimbu-v2.005 Date: 2024-06-09

Font version

Version 2.005

Issue

The space added in 'kern' for spacing marks like U+1929 LIMBU SUBJOINED LETTER YA and U+1921 LIMBU VOWEL SIGN I is cumulative: if a cluster has multiple of those spacing marks, too much advance width is added. It should only add enough width for the widest of the marks.

Character data

ᤁᤩᤡᤁ U+1901 LIMBU LETTER KA U+1929 LIMBU SUBJOINED LETTER YA U+1921 LIMBU VOWEL SIGN I U+1901 LIMBU LETTER KA

Screenshot

ᤁᤩᤡᤁ

simoncozens commented 1 week ago

I've added a hack for this (just add spacing for the first spacing mark), but it doesn't feel great. If there are plausible clusters which have a second spacing mark wider than the fix, I can fix it up.

dscorbett commented 1 week ago

The sequence <consonant, post-base consonant sign, post-base vowel sign, consonant> is plausible, but in the latest dev build, <U+1901, U+192B, U+1923, U+1901> ⟨ᤁᤫᤣᤁ⟩ looks like this:

dscorbett commented 1 week ago

The sequence <post-base consonant sign, U+1921, U+193A> seems plausible too. Here is <U+1901, U+1929, U+1921, U+193A, U+1901> ⟨ᤁᤩᤡ᤺ᤁ⟩:

notofonts / limbu