removeChildren() remove only first level children. close #135

CHANGELOG.md

@@ -10,6 +10,8 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 - `Konva.Path` will be filled when it is not closed
 - `Animation.start()` will not not immediate sync draw. This should improve performance a little.
 - Warning when node for `Tween` is not in layer yet.
+- `removeChildren()` remove only first level children. So it will not remove grandchildren.
+
 
 ## [0.12.4][2016-04-19]
 

konva.js

@@ -6642,9 +6642,6 @@
                 // reset parent to prevent many _setChildrenIndices calls
                 delete child.parent;
                 child.index = 0;
-                if (child.hasChildren()) {
-                    child.removeChildren();
-                }
                 child.remove();
             }
             children = null;

src/Container.js

@@ -69,9 +69,6 @@
                 // reset parent to prevent many _setChildrenIndices calls
                 delete child.parent;
                 child.index = 0;
-                if (child.hasChildren()) {
-                    child.removeChildren();
-                }
                 child.remove();
             }
             children = null;

test/unit/Container-test.js

@@ -528,7 +528,7 @@ suite('Container', function() {
         layer.draw();
 
         assert.equal(layer.children.length, 0, 'layer should have 0 children');
-        assert.equal(group.children.length, 0, 'group should have 0 children');
+        assert.equal(group.children.length, 2, 'group still should have 2 children');
     });
 
     // ======================================================