Update category names, allow descriptions (#853)

* Update category names, allow descriptions * Don't require ufomerge for this test * Don't build variable, it requires hb-subset
googlefonts · Mar 13, 2024 · 5e1cb5c · 5e1cb5c
1 parent ddb743f
commit 5e1cb5c
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 15 deletions.
diff --git a/Lib/gftools/tags.py b/Lib/gftools/tags.py
@@ -61,6 +61,7 @@ class GFTags(object):
             "Medieval",
             "Brush/Marker",
             "Pixel",
+            "Brush",
         ],
         "Arabic": [
             "Kufi",
@@ -82,15 +83,15 @@ class GFTags(object):
             "Chrieng (Khmer)",
         ],
         "Sinhala": [
-            "Traditional/High contrast",
-            "Contemporary/High contrast",
+            "Traditional",
+            "Contemporary",
             "Low contrast",
         ],
         "Indic": [
-            "Traditional/High contrast",
-            "Contemporary/High contrast",
+            "Traditional",
+            "Contemporary",
             "Low contrast",
-            "Sign Painting/vernacular",
+            "Sign Painting",
             "Reverse-contrast",
         ],
         "Simplicity": [
@@ -130,6 +131,29 @@ class GFTags(object):
             "Emojis",
             "Symbols",
         ],
+        "Expressive": [
+            "Business",
+            "Sincere",
+            "Loud",
+            "Vintage",
+            "Calm",
+            "Calm/simple",
+            "Stiff",
+            "Competent",
+            "Happy",
+            "Childlike",
+            "Excited",
+            "Playful",
+            "Awkward",
+            "Innovative",
+            "Rugged",
+            "Futuristic",
+            "Artistic",
+            "Cute",
+            "Fancy",
+            "Sophisticated",
+            "Active",
+        ]
     }
 
     def __init__(self):
@@ -164,7 +188,7 @@ def _parse_csv(self, data, skip_rows=[], skip_columns=[], family_name_col=0):
                 # If no tag exists for a value, it means a value has been assigned
                 # to the whole group such as Sans, Sans Serif etc. We don't want to
                 # include these since we can deduce it ourselves according to Evan.
-                sub_category = data[1][j]
+                sub_category = data[1][j].split("\n")[0]
                 if sub_category == "":
                     continue
                 if category not in self.CATEGORIES:

diff --git a/data/test/builder/recipeprovider_noto/config.yaml b/data/test/builder/recipeprovider_noto/config.yaml
@@ -5,10 +5,4 @@ autohintTTF: false
 autohintOTF: false
 recipeProvider: noto
 googleFonts: true
-includeSubsets:
-- from: Noto Sans Devanagari
-  layoutHandling: closure
-  force: true
-  ranges:
-    - start: 0x900
-      end: 0x97F
+buildVariable: false
diff --git a/tests/test_builder.py b/tests/test_builder.py
@@ -49,8 +49,8 @@
         (
             os.path.join(TEST_DIR, "recipeprovider_noto"),
             [
-                os.path.join("TestFamily", "unhinted", "slim-variable-ttf", "TestFamily[wght].ttf"),
-                os.path.join("TestFamily", "googlefonts", "variable", "TestFamily[wght].ttf")
+                os.path.join("TestFamily", "unhinted", "ttf", "TestFamily-Regular.ttf"),
+                os.path.join("TestFamily", "googlefonts", "ttf", "TestFamily-Black.ttf"),
             ],
         )
     ],