search

kiliman / operator-mono-lig

Add ligatures to Operator Mono similar to Fira Code
MIT License
3.2k stars 202 forks source link

OperatorMonoSSm-Medium is missing ligatures #77

Closed jtourlamain closed 5 years ago

jtourlamain commented 5 years ago

I was able to build the OperatorMonoSSm-Book font, but the OperatorMonoSSm-Medium gives warnings during build. Ligatures like "->" are missing in the font.

On WSL I used 

Building OperatorMonoSSmLig-Medium
Dumping "./original/OperatorMonoSSm-Medium.otf" to "./original/OperatorMonoSSm-Medium.ttx"...
Dumping 'GlyphOrder' table...
Dumping 'head' table...
Dumping 'hhea' table...
Dumping 'maxp' table...
Dumping 'OS/2' table...
Dumping 'name' table...
Dumping 'cmap' table...
Dumping 'post' table...
Dumping 'CFF ' table...
Dumping 'GPOS' table...
Dumping 'GSUB' table...
Dumping 'hmtx' table...
Dumping 'DSIG' table...
Reading original font file ./original/OperatorMonoSSm-Medium.ttx
Patching names
Patching glyphs
Patching gpos
Patching gsub
Patching hmtx
Patching charstrings
Writing ligature font file ./build/OperatorMonoSSmLig-Medium.ttx
Done
Compiling "./build/OperatorMonoSSmLig-Medium.ttx" to "./build/OperatorMonoSSmLig-Medium.otf"...
Parsing 'GlyphOrder' table...
Parsing 'head' table...
Parsing 'hhea' table...
Parsing 'maxp' table...
Parsing 'OS/2' table...
Parsing 'name' table...
Parsing 'cmap' table...
Parsing 'post' table...
Parsing 'CFF ' table...
Parsing 'GPOS' table...
Parsing 'GSUB' table...
Parsing 'hmtx' table...
Parsing 'DSIG' table...
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.
WARNING: GSUB/GPOS Coverage is not sorted by glyph ids.

end of output in OperatorMonoSSmLig-Medium.ttx:

<DSIG>
        <!-- note that the Digital Signature will be invalid after recompilation! -->
        <tableHeader flag="0x0" numSigs="0" version="1"/>
    </DSIG>
kiliman commented 5 years ago

Depending on the editor you use, some ligatures won't work, especially those that start with a hyphen.

See https://github.com/tonsky/FiraCode/issues/259 and https://github.com/tonsky/FiraCode/issues/422 for details.

jtourlamain commented 5 years ago

@kiliman thanks for your quick feedback. I didn't know it was editor dependend. I thought that all the warnings meant that some ligatures where missing during build