Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix missing moveTos in SVG paths

Browse Source
This commit is contained in:
Jani Pehkonen 2019-02-26 20:00:35 +02:00
parent 652059056c
commit 52e8e9b059
1 changed files with 18 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/display/svg.js
View File

@ -940,7 +940,6 @@ SVGGraphics = (function SVGGraphicsClosure() {
constructPath: function SVGGraphics_constructPath(ops, args) {
var current = this.current;
var x = current.x, y = current.y;
current.path = this.svgFactory.createElement('svg:path');
var d = [];
var opLength = ops.length;
@ -992,10 +991,22 @@ SVGGraphics = (function SVGGraphicsClosure() {
break;
}
}
current.path.setAttributeNS(null, 'd', d.join(' '));
current.path.setAttributeNS(null, 'fill', 'none');
this._ensureTransformGroup().appendChild(current.path);
d = d.join(' ');
if (current.path && opLength > 0 && ops[0] !== OPS.rectangle &&
ops[0] !== OPS.moveTo) {
// If a path does not start with an OPS.rectangle or OPS.moveTo, it has
// probably been divided into two OPS.constructPath operators by
// OperatorList. Append the commands to the previous path element.
d = current.path.getAttributeNS(null, 'd') + d;
} else {
current.path = this.svgFactory.createElement('svg:path');
this._ensureTransformGroup().appendChild(current.path);
}
current.path.setAttributeNS(null, 'd', d);
current.path.setAttributeNS(null, 'fill', 'none');
// Saving a reference in current.element so that it can be addressed
// in 'fill' and 'stroke'
@ -1004,6 +1015,9 @@ SVGGraphics = (function SVGGraphicsClosure() {
},
endPath: function SVGGraphics_endPath() {
// Painting operators end a path.
this.current.path = null;
if (!this.pendingClip) {
return;
}