Broken remove on two

[xixixao]

Adding

two.remove(circle);

on line 15 of the first example and pressing enter couple times breaks havoc (rectangle is rendered without fill and on each redraw the scene is shifted to the right).

Secondly, is there any way to completely wipe Two's instance and start anew? Unfortunately, now Two is unusable in situations when I want to draw dynamically, different objects into the same location. Or something like two.clear().

[jonobr1]

Thanks for mentioning this. Remove was one of the latest things I've put in and as such is probably the buggiest. I'll look into this right away.

[jonobr1]

So on further inspection.two.remove(circle); doesn't exist in the API. The way to remove the circle would be to circle.remove();. I will add a two.remove method so that you can do what you'd like. If you'd like to remove all shapes on a page I suggest:

_.each(two.scene.children, function(child) {
  child.remove();
});

Be mindful of removing any references in your own code to shapes and groups in order for the browser to properly garbage collect these.

[xixixao]

Would be good to document this use somewhere in the API, the two descriptions of group.remove() are not very clear to me. Thanks for the snippet, I think this could be result of two.scene.clear().

[jonobr1]

No problem. Indeed — I think I'm gonna craft the api like that. I can update this on the documentation, but yes, remove is odd and takes lineage from backbone's bind and unbind. i.e:

group.remove(circle); // Removes the circle
group.remove(circle, rect); // Removes circle and rect
group.remove([circle, rect]); // Removes circle and rect
group.remove(); // Removes the group itself

[jonobr1]

Added and in the latest build: a5c08c4