fix bug & update preview

README.md

@@ -28,7 +28,7 @@ Both offset/offsetStroke take the form of **f(offset: number, options?: {})**, t
 <br/>&nbsp;&nbsp;**insert**: whether the result should be insert into the canvas, default is **true**.
 
 ## Preview
-There are some cases that the library may return weird result or failed silently, please let me noticed in the project issues. And in some cases the library will yeild an ok result than a perfect one.
+There are some cases that the library may return weird result or failed silently, please let me noticed in the project issues. And in some cases the library will yeild an ok result than a perfect one. Currently the library should give good results for closed shapes, but may fail in some open curve cases, I'm still working on it.
 ![Preview](/public/preview.jpg)
 
 ## Development

demo/paperjs-offset.js

@@ -449,11 +449,6 @@
                 newPath.children.filter(function (c) {
                     if (c.segments.length > 1) {
                         return !IsSameDirection(c, path);
-                        // let sample1 = c.segments[0].location.offset
-                        // let sample2 = c.segments[Math.max(1, Math.floor(c.segments.length / 2))].location.offset
-                        // let offset1 = path.getNearestLocation(c.getPointAt((sample1 + sample2) / 3)).offset
-                        // let offset2 = path.getNearestLocation(c.getPointAt((sample1 + sample2) / 3 * 2)).offset
-                        // return offset1 > offset2
                     }
                     else {
                         return true;
@@ -596,6 +591,7 @@
             result = new paper.CompoundPath({ children: children, insert: false });
         }
         result.copyAttributes(nonSIPath, false);
+        result.remove();
         return result;
     }
     function OffsetStroke(path, offset, join, cap, limit) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "paperjs-offset",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "An offset function for paperjs path.",
   "main": "dist/index.umd.js",
   "module": "dist/index.es5.js",

src/Offset.ts

@@ -191,11 +191,6 @@ namespace Offsets {
       (newPath.children as Array<paper.Path>).filter(c => {
         if (c.segments.length > 1) {
           return !IsSameDirection(c, path)
-          // let sample1 = c.segments[0].location.offset
-          // let sample2 = c.segments[Math.max(1, Math.floor(c.segments.length / 2))].location.offset
-          // let offset1 = path.getNearestLocation(c.getPointAt((sample1 + sample2) / 3)).offset
-          // let offset2 = path.getNearestLocation(c.getPointAt((sample1 + sample2) / 3 * 2)).offset
-          // return offset1 > offset2
         } else {
           return true
         }
@@ -335,6 +330,7 @@ export function OffsetPath(path: PathType, offset: number, join: StrokeJoinType,
     result = new paper.CompoundPath({ children, insert: false })
   }
   result.copyAttributes(nonSIPath, false)
+  result.remove()
   return result
 }