Broken remove on two about two.js HOT 5 CLOSED

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.

from two.js.

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.

from two.js.

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().

from two.js.

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

from two.js.

Added and in the latest build: a5c08c4

from two.js.