search

googlefonts / fontc

Where in we pursue oxidizing (context: https://github.com/googlefonts/oxidize) fontmake.
Apache License 2.0
76 stars 13 forks source link

[glyphs] Add test for anchor propagation on digraph #790

Closed cmyr closed 4 months ago

cmyr commented 4 months ago

This is just to sanity check that our behaviour is matching glyphs.app in the case where we have a glyph with multiple base components but which is not a ligature.

JMM

moyogo commented 4 months ago

This seems wrong whether Glyphs.app is doing it or not.

cmyr commented 4 months ago

@moyogo do you mean in the sense that the generated anchors don't really make sense? I agree that the font designer should probably be either adding anchors manually or else opting out of propagation here if possible, but we definitely don't want these glyphs to be treated as ligatures.

In any case we are defining 'correct' here in terms of what glyphs.app does, and this is correct by that measure.

moyogo commented 4 months ago

@cmyr I commented on https://github.com/googlefonts/glyphsLib/issues/368#issuecomment-2103445759