Merge pull request #2793 from HackbrettXXX/context2d-autopaging

fix context2d autoPaging with 10 or more pages
parallax · Jul 2, 2020 · 08754f3 · 08754f3
2 parents f30e68f + 1da7e56
commit 08754f3
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 3 deletions.
diff --git a/spec/context2d.spec.js b/spec/context2d.spec.js
diff --git a/spec/reference/autoPaging10Pages.pdf b/spec/reference/autoPaging10Pages.pdf
diff --git a/src/modules/context2d.js b/src/modules/context2d.js
@@ -1441,7 +1441,7 @@
       }
     }
 
-    pages.sort();
+    sortPages(pages);
 
     var clipPath;
     if (this.autoPaging) {
@@ -1595,6 +1595,12 @@
     return paths;
   };
 
+  var sortPages = function (pages) {
+    return pages.sort(function (a, b) {
+      return a - b;
+    })
+  }
+
   var pathPreProcess = function(rule, isClip) {
     var fillStyle = this.fillStyle;
     var strokeStyle = this.strokeStyle;
@@ -1625,7 +1631,7 @@
         addPage.call(this);
       }
     }
-    pages.sort();
+    sortPages(pages);
 
     if (this.autoPaging) {
       var min = pages[0];
@@ -1987,7 +1993,7 @@
       }
     }
 
-    pages.sort();
+    sortPages(pages);
 
     var clipPath, oldSize;
     if (this.autoPaging === true) {