Improve the handling of the connection order

Author: flekschas

CHANGELOG.md

@@ -1,5 +1,11 @@
 ## Next
 
+**Breaking changes:**
+
+- Improve the handling of the connection order by switching from a structured 5th component to a 6th component.
+
+  Previously to provide a specific order, the 5th component had to be a tuple of `[lineComponentId, orderIndex]`. With this update the 5th component will always just be `lineComponentId` and the 6th will be `orderIndex`.
+
 ## v0.18.7
 
 - Avoid text selection when lassoing

example/connected-points.js

@@ -85,6 +85,7 @@ const generatePoints = (num) => {
           g, // category
           Math.random(), // value
           g * numPointsPerStep + i, // to identify connected points
+          // (s + 2) % 5, // specifies the order of the connected points
         ]);
       }
     }

src/spline-curve-worker.js

@@ -210,13 +210,13 @@ const worker = function worker() {
   const groupPoints = (points) => {
     const groupedPoints = {};
 
+    const isOrdered = !Number.isNaN(+points[0][5]);
     points.forEach((point) => {
-      const isStruct = Array.isArray(point[4]);
-      const segId = isStruct ? point[4][0] : point[4];
+      const segId = point[4];
 
       if (!groupedPoints[segId]) groupedPoints[segId] = [];
 
-      if (isStruct) groupedPoints[segId][point[4][1]] = point;
+      if (isOrdered) groupedPoints[segId][point[5]] = point;
       else groupedPoints[segId].push(point);
     });