build_stat: improve elided axisvalues

diff --git a/Lib/axisregistry/__init__.py b/Lib/axisregistry/__init__.py
index 7617da0554b7cae555d7a8b555a860dd0e90067b..7d2ccf68a0bb473c501b15543d1605dc108b4b18 100644 (file)
--- a/Lib/axisregistry/__init__.py
+++ b/Lib/axisregistry/__init__.py
@@ -234,14 +234,17 @@ def build_stat(ttFont, sibling_ttFonts=[]):
     for axis, fallback in fallbacks_in_siblings:
         if axis in seen_axes:
             continue
-        value = 0.0
+        elided_value = axis_registry[axis].default_value
+        elided_fallback = axis_registry.fallback_for_value(axis, elided_value)
         a = {
             "tag": axis,
             "name": axis_registry[axis].display_name,
-            "values": [{"name": "Normal", "value": value, "flags": 0x2}],
+            "values": [
+                {"name": elided_fallback.name, "value": elided_value, "flags": 0x2}
+            ],
         }
-        if axis in LINKED_VALUES and value in LINKED_VALUES[axis]:
-            a["values"][0]["linkedValue"] = LINKED_VALUES[axis][value]
+        if axis in LINKED_VALUES and elided_value in LINKED_VALUES[axis]:
+            a["values"][0]["linkedValue"] = LINKED_VALUES[axis][elided_value]
         res.append(a)
     buildStatTable(ttFont, res, macNames=False)
 

diff --git a/tests/data/OpenSans-Italic[wdth,wght]_STAT.ttx b/tests/data/OpenSans-Italic[wdth,wght]_STAT.ttx
index 6b82b0fcfd010438d96b952b25d029f7653a3979..0cd24334252fd12c4777e5208dbd64eddac990fc 100644 (file)
--- a/tests/data/OpenSans-Italic[wdth,wght]_STAT.ttx
+++ b/tests/data/OpenSans-Italic[wdth,wght]_STAT.ttx
@@ -89,7 +89,7 @@
   <AxisValue index="10" Format="3">
     <AxisIndex value="3"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="288"/>  <!-- Normal -->
+    <ValueNameID value="289"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>

diff --git a/tests/data/OpenSansCondensed-Italic[wght]_STAT.ttx b/tests/data/OpenSansCondensed-Italic[wght]_STAT.ttx
index d0e9f35ca31a3ce8a6f1753307ebbc3a918ce83b..8c3c2b9f18abfb0a7dd4a66ca24a074595d2d874 100644 (file)
--- a/tests/data/OpenSansCondensed-Italic[wght]_STAT.ttx
+++ b/tests/data/OpenSansCondensed-Italic[wght]_STAT.ttx
@@ -94,15 +94,15 @@
   <AxisValue index="10" Format="3">
     <AxisIndex value="3"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="529"/>  <!-- Normal -->
+    <ValueNameID value="529"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>
   <AxisValue index="11" Format="1">
     <AxisIndex value="4"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="529"/>  <!-- Normal -->
-    <Value value="0.0"/>
+    <ValueNameID value="530"/>  <!-- Normal -->
+    <Value value="100.0"/>
   </AxisValue>
 </AxisValueArray>
 <ElidedFallbackNameID value="2"/>  <!-- Italic -->
\ No newline at end of file

diff --git a/tests/data/OpenSansCondensed[wght]_STAT.ttx b/tests/data/OpenSansCondensed[wght]_STAT.ttx
index db4d34c738523e1e12152a6a763214a5a3a9eea9..465a86ccfe8cb7c0daea71eb5dec1673ab0d6cfb 100644 (file)
--- a/tests/data/OpenSansCondensed[wght]_STAT.ttx
+++ b/tests/data/OpenSansCondensed[wght]_STAT.ttx
@@ -88,7 +88,7 @@
   <AxisValue index="9" Format="3">
     <AxisIndex value="2"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="280"/>  <!-- Normal -->
+    <ValueNameID value="466"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>
@@ -96,12 +96,12 @@
     <AxisIndex value="3"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
     <ValueNameID value="280"/>  <!-- Normal -->
-    <Value value="0.0"/>
+    <Value value="100.0"/>
   </AxisValue>
   <AxisValue index="11" Format="3">
     <AxisIndex value="4"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="280"/>  <!-- Normal -->
+    <ValueNameID value="466"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>

diff --git a/tests/data/OpenSans[wdth,wght]_STAT.ttx b/tests/data/OpenSans[wdth,wght]_STAT.ttx
index 7ab8b63e42333e36fd1760d7d56e16eb9706409c..d41cf388930f4f3f3dabdf4aaf032f58a74818d1 100644 (file)
--- a/tests/data/OpenSans[wdth,wght]_STAT.ttx
+++ b/tests/data/OpenSans[wdth,wght]_STAT.ttx
@@ -83,14 +83,14 @@
   <AxisValue index="9" Format="3">
     <AxisIndex value="2"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="283"/>  <!-- Normal -->
+    <ValueNameID value="285"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>
   <AxisValue index="10" Format="3">
     <AxisIndex value="3"/>
     <Flags value="2"/>  <!-- ElidableAxisValueName -->
-    <ValueNameID value="283"/>  <!-- Normal -->
+    <ValueNameID value="285"/>  <!-- Roman -->
     <Value value="0.0"/>
     <LinkedValue value="1.0"/>
   </AxisValue>

diff --git a/tests/test_names.py b/tests/test_names.py
index 1a907967db493f718d70722b7d4568a603f99c7b..18543a38b0dd49b7c276a9f0a7157dee73352d6d 100644 (file)
--- a/tests/test_names.py
+++ b/tests/test_names.py
@@ -448,7 +448,7 @@ def test_stat(fp, sibling_fps):
     build_stat(font, siblings)
     stat_fp = fp.replace(".ttf", "_STAT.ttx")
 
-    ### output good files
+    ## output good files
     # with open(stat_fp, "w") as doc:
     #    got = dump(font["STAT"], font)
     #    doc.write(got)