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

[interpolatable] Try drawing kink point at wrong-contour-order

Browse Source 
I'm not sure it's correct, but doesn't seem to harm.
This commit is contained in:
Behdad Esfahbod 2023-11-26 22:11:31 -07:00
parent c3213e20bb
commit b5f022c5fd
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
Lib/fontTools/varLib/interpolatablePlot.py
View File

@ -496,10 +496,6 @@ class InterpolatablePlot:
scale=min(scales), scale=min(scales),
) )
if "kink" in problem_types:
pen = PerContourOrComponentPen(RecordingPen)
midway_glyphset[glyphname].draw(pen)
y += self.height + self.pad y += self.height + self.pad
# Draw the proposed fix # Draw the proposed fix
@ -570,6 +566,8 @@ class InterpolatablePlot:
# Replace the wrong contours # Replace the wrong contours
wrongContour1.value = segment1.value wrongContour1.value = segment1.value
wrongContour2.value = segment2.value wrongContour2.value = segment2.value
perContourPen1.value[problem["contour"]] = wrongContour1
perContourPen2.value[problem["contour"]] = wrongContour2
for problem in problems: for problem in problems:
# If we have a kink, try to fix it. # If we have a kink, try to fix it.
@ -951,6 +949,15 @@ class InterpolatablePlot:
perContourPen.value[idx if matching is None else matching[idx]].replay( perContourPen.value[idx if matching is None else matching[idx]].replay(
converter converter
) )
if which == 1 or midway:
wrong_start_point_problem = [pt for pt in problems if pt["type"] == "wrong_start_point" and pt.get("contour") == idx]
if wrong_start_point_problem:
proposed_start = wrong_start_point_problem[0]["value_2"]
points.value = points.value[proposed_start:] + points.value[:proposed_start]
if wrong_start_point_problem[0]["reversed"]:
points.value = points.value[::-1]
targetPoint = points.value[problem["value"]][0] targetPoint = points.value[problem["value"]][0]
cr.save() cr.save()
cr.translate(*targetPoint) cr.translate(*targetPoint)