JulietaUla / Montserrat

SIL Open Font License 1.1
1.62k stars 245 forks source link

SSA suggested fixes #149

Closed moyogo closed 4 days ago

moyogo commented 1 week ago

There a few missing anchors and a couple of designs that could be adjusted.

uniA7CC uniA7CD

uniA7CC uniA7CD (Ꟍꟍ) should have slash strokes.

Current Proposed
Screenshot 2024-09-26 at 22 37 54 Screenshot 2024-09-26 at 22 37 45
Screenshot 2024-09-26 at 22 42 46 Screenshot 2024-09-26 at 22 42 36

Upsilon-latin upsilon-latin

This vowel is used with combining diacritics.

Current Proposed
Screenshot 2024-09-26 at 22 46 28 Screenshot 2024-09-26 at 22 46 17
Screenshot 2024-09-26 at 22 46 50 Screenshot 2024-09-26 at 22 46 04

uniA7DC lambdastroke

uniA7DC (Lambdastroke) and lambdastroke are used with commaabovecomb The anchors can also be added to uniA7DA uniA7DB (Lambda-latin lambda-latin).

Current Proposed
Screenshot 2024-09-26 at 22 57 51 Screenshot 2024-09-26 at 22 58 11

Eopen eopen Oopen oopen

Eopen eopen Oopen oopen are sometimes used with ogonekcomb.

Current Proposed
Screenshot 2024-09-26 at 22 58 58 Screenshot 2024-09-26 at 22 59 05
Screenshot 2024-09-26 at 23 05 01 Screenshot 2024-09-26 at 23 04 44

Iota-latin iota-latin

Iota-latin iota-latin are sometimes used with ogonekcomb. Match the ogonek anchor in all their glyphs.

Current Proposed
Screenshot 2024-09-26 at 23 00 13 Screenshot 2024-09-26 at 23 00 23

Vturned vturned

Vturned vturned is sometimes used with ogonek.

Current Proposed
Screenshot 2024-09-26 at 23 02 09 Screenshot 2024-09-26 at 23 02 17

acutecomb.case, circumflexcomb.case, etc.

Anchors in Montserrat and Montserrat-Italic should match some .case combining marks are missing top anchors or have it low in Montserrat compared to Montserrat-Italic.

Current Proposed
Screenshot 2024-09-26 at 23 06 59 Screenshot 2024-09-26 at 23 09 20
juandelperal commented 4 days ago

Thanks @moyogo for your contribution, is now merged.

Thanks for adding the missing anchors. We will review some details in the merged branch