Merge pull request #964 from googlefonts/skip-ignorable-anchors

markFeatureWriter: Skip ignorable anchors
googlefonts · Nov 23, 2023 · 17aed70 · 17aed70
2 parents dc4354b + c5d975d
commit 17aed70
Show file tree

Hide file tree

Showing 3 changed files with 434 additions and 2 deletions.
diff --git a/Lib/glyphsLib/featureWriters/markFeatureWriter.py b/Lib/glyphsLib/featureWriters/markFeatureWriter.py
@@ -24,6 +24,7 @@ class ContextuallyAwareNamedAnchor(NamedAnchor):
         "number",
         "markClass",
         "isContextual",
+        "isIgnorable",
         "libData",
     )
 
@@ -83,13 +84,15 @@ def parseAnchorName(
         else:
             isMark = False
 
-        return isMark, key, number, isContextual
+        isIgnorable = not key[0].isalpha()
+
+        return isMark, key, number, isContextual, isIgnorable
 
     def __init__(self, name, x, y, markClass=None, libData=None):
         self.name = name
         self.x = x
         self.y = y
-        isMark, key, number, isContextual = self.parseAnchorName(
+        isMark, key, number, isContextual, isIgnorable = self.parseAnchorName(
             name,
             markPrefix=self.markPrefix,
             ligaSeparator=self.ligaSeparator,
@@ -106,6 +109,7 @@ def __init__(self, name, x, y, markClass=None, libData=None):
         self.number = number
         self.markClass = markClass
         self.isContextual = isContextual
+        self.isIgnorable = isIgnorable
         self.libData = libData
 
 
@@ -144,6 +148,8 @@ def _getAnchorLists(self):
                 a = self.NamedAnchor(name=anchorName, x=x, y=y, libData=libData)
                 if a.isContextual and not libData:
                     continue
+                if a.isIgnorable:
+                    continue
                 anchorDict[anchorName] = a
             if anchorDict:
                 result[glyphName] = list(anchorDict.values())