shadowfacts/fonttools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

TupleVariation.toXML: sort attrs min/value/max in TTX dump

Browse Source 
It's more legible if the peak value is written after the min and before the max value
defining a given region.
This commit is contained in:
Cosimo Lupo 2019-03-04 16:41:26 -08:00
parent 694b57754c
commit 71e44a52ee
4 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
Lib/fontTools/ttLib/tables/TupleVariation.py
View File

@ -71,7 +71,13 @@ class TupleVariation(object):
if minValue == defaultMinValue and maxValue == defaultMaxValue:
writer.simpletag("coord", axis=axis, value=value)
else:
writer.simpletag("coord", axis=axis, value=value, min=minValue, max=maxValue)
attrs = [
("axis", axis),
("min", minValue),
("value", value),
("max", maxValue),
]
writer.simpletag("coord", attrs)
writer.newline()
wrote_any_deltas = False
for i, delta in enumerate(self.coordinates):

6
Tests/ttLib/tables/TupleVariation_test.py
View File

@ -107,7 +107,7 @@ class TupleVariationTest(unittest.TestCase):
self.assertIn("bad delta format", [r.msg for r in captor.records])
self.assertEqual([
'<tuple>',
'<coord axis="wdth" max="0.5" min="0.3" value="0.4"/>',
'<coord axis="wdth" min="0.3" value="0.4" max="0.5"/>',
'<!-- bad delta #0 -->',
'</tuple>',
], TupleVariationTest.xml_lines(writer))
@ -118,7 +118,7 @@ class TupleVariationTest(unittest.TestCase):
g.toXML(writer, ["wdth", "wght", "opsz"])
self.assertEqual([
'<tuple>',
'<coord axis="wdth" max="0.5" min="0.3" value="0.4"/>',
'<coord axis="wdth" min="0.3" value="0.4" max="0.5"/>',
'<coord axis="wght" value="1.0"/>',
'<coord axis="opsz" value="-0.7"/>',
'<delta cvt="0" value="42"/>',
@ -134,7 +134,7 @@ class TupleVariationTest(unittest.TestCase):
g.toXML(writer, ["wdth", "wght", "opsz"])
self.assertEqual([
'<tuple>',
'<coord axis="wdth" max="0.5" min="0.3" value="0.4"/>',
'<coord axis="wdth" min="0.3" value="0.4" max="0.5"/>',
'<coord axis="wght" value="1.0"/>',
'<coord axis="opsz" value="-0.7"/>',
'<delta pt="0" x="9" y="8"/>',

2
Tests/ttLib/tables/_g_v_a_r_test.py
View File

@ -93,7 +93,7 @@ GVAR_XML = [
'</glyphVariations>',
'<glyphVariations glyph="I">',
' <tuple>',
' <coord axis="wght" max="1.0" min="0.0" value="0.5"/>',
' <coord axis="wght" min="0.0" value="0.5" max="1.0"/>',
' <delta pt="0" x="3" y="3"/>',
' <delta pt="1" x="1" y="1"/>',
' <delta pt="2" x="4" y="4"/>',

4
Tests/varLib/data/test_results/SparseMasters.ttx
View File

@ -574,7 +574,7 @@
</glyphVariations>
<glyphVariations glyph="e">
<tuple>
<coord axis="wght" max="1.0" min="0.0" value="0.36365"/>
<coord axis="wght" min="0.0" value="0.36365" max="1.0"/>
<delta pt="0" x="-1" y="-25"/>
<delta pt="1" x="0" y="0"/>
<delta pt="2" x="-84" y="5"/>
@ -594,7 +594,7 @@
<delta pt="16" x="0" y="0"/>
</tuple>
<tuple>
<coord axis="wght" max="1.0" min="0.36365" value="1.0"/>
<coord axis="wght" min="0.36365" value="1.0" max="1.0"/>
<delta pt="0" x="70" y="1"/>
<delta pt="1" x="70" y="1"/>
<delta pt="2" x="-76" y="1"/>