fix: correctly generate spacers

mishamyrt · Aug 2, 2024 · 7f8806e · 7f8806e
1 parent 66704a6
commit 7f8806e
Show file tree

Hide file tree

Showing 3 changed files with 940 additions and 346 deletions.
diff --git a/scripts/font.py b/scripts/font.py
@@ -102,7 +102,6 @@ def set_features(font: GlyphsFont, cls: list[str], fea: list[GSFeature]):
     features = fea.copy()
     calt = generate_calt(font)
     features.append(calt)
-    generate_spacers(font.ligatures(), font.file.glyphs)
     font.set_classes(cls)
     font.set_features(features)
 
@@ -116,6 +115,7 @@ def load_font(args):
     for file in config_file["family"]:
         font = GlyphsFont(os.path.join(source_dir, file))
         config["fonts"].append(font)
+        generate_spacers(font.ligatures(), font.file.glyphs)
         if config_file["family"][file] == "all":
             set_features(font, cls, fea)
     return args, config

diff --git a/scripts/liblilex/generator/spacers.py b/scripts/liblilex/generator/spacers.py
@@ -13,6 +13,8 @@ def generate_spacers(ligatures: list[str], glyphs: list[GSGlyph]) -> list[str]:
 
     unique = list(set(unique))
     template = glyphs["spacer.tpl"]
+    if template is None:
+        raise ValueError("Missing 'spacer.tpl' in glyphs font")
     for glh in unique:
         name = f"{glh}.spacer"
         if name not in glyphs: