Merge pull request #1981 from googlefonts/bugfix-delete-points-issue-…

…1980 Fix bug with deleting multiple segments, one of which includes the starting point
googlefonts · Jan 23, 2025 · 4bf5d24 · 4bf5d24
2 parents 7e209f2 + 8d9df9e
commit 4bf5d24
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/src/fontra/client/core/path-functions.js b/src/fontra/client/core/path-functions.js
@@ -425,12 +425,8 @@ export function deleteSelectedPoints(path, pointIndices) {
         newPoints = computeHandlesFromFragment(curveType, fragment.contour);
       } else {
         newPoints = [];
-        path.setPointType(indices[0] + startPoint + indexBias, points[0].type, false);
-        path.setPointType(
-          indices.at(-1) + startPoint + indexBias,
-          points.at(-1).type,
-          false
-        );
+        path.setPointType(indices[0] + startPoint + 0, points[0].type, false);
+        path.setPointType(indices.at(-1) + startPoint + 0, points.at(-1).type, false);
       }
       for (const newPoint of newPoints) {
         path.insertPoint(contourIndex, insertIndex, newPoint);