Skip to content

Commit cfa5374

Browse files
committed
Merge pull request processing-js#42 from Pomax/t1930
turns out SVG child parsing added an extra child step between all elements
2 parents b2013fb + 24de2db commit cfa5374

File tree

5 files changed

+363
-14
lines changed

5 files changed

+363
-14
lines changed

processing.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3949,14 +3949,17 @@
39493949
*/
39503950
PShapeSVG.prototype.parseChildren = function(element) {
39513951
var newelement = element.getChildren();
3952-
var children = new p.PShape();
3953-
for (var i = 0, j = newelement.length; i < j; i++) {
3952+
var base = new p.PShape();
3953+
var i, j;
3954+
for (i = 0, j = newelement.length; i < j; i++) {
39543955
var kid = this.parseChild(newelement[i]);
39553956
if (kid) {
3956-
children.addChild(kid);
3957+
base.addChild(kid);
39573958
}
39583959
}
3959-
this.children.push(children);
3960+
for (i = 0, j = base.children.length; i < j; i++) {
3961+
this.children.push(base.children[i]);
3962+
}
39603963
};
39613964
/**
39623965
* @member PShapeSVG

test/ref/loadshape-child-nesting.pde

Lines changed: 17 additions & 0 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)