Dropping descendants behaviour improvement, visual improvements

Author: nikitaeverywhere

package.json

@@ -1,6 +1,6 @@
 {
   "name": "iknow-entity-browser",
-  "version": "0.4.1",
+  "version": "0.4.2",
   "description": "Visualizer for iKnow entities",
   "main": "gulpfile.babel.js",
   "scripts": {

src/static/js/controls.js

@@ -14,9 +14,12 @@ onSelectionUpdate((sel) => {
 });
 
 function updateButtons () {
-    let display = selection.length ? "block" : "none";
-    dropChildrenButton.style.display = display;
+    let toDrop = 0;
+    for (let node of selection) {
+        toDrop += (node.children ? node.children.length : 0);
+    }
     removeButton.classList.add("disabled"); // temporary
+    dropChildrenButton.classList[toDrop > 0 ? "remove" : "add"]("disabled");
 }
 
 function deleteSelection () {

src/static/js/model/index.js

@@ -277,24 +277,21 @@ export function dropDescendants (nodes) {
 
     let toDrop = 0;
     for (let node of nodes) {
-        toDrop += (node.children ? node.children.length : 0)
-            + (node._children ? node._children.length : 0);
+        toDrop += (node.children ? node.children.length : 0);
     }
     if (toDrop === 0)
         return 0;
 
     let restore = nodes.slice().map(node => {
         return {
             node: node,
-            children: node.children,
-            _children: node._children
+            children: node.children
         };
     });
 
     function f () {
         for (let node of nodes) {
             node.children = [];
-            node._children = [];
         }
         dataUpdated();
     }
@@ -305,7 +302,6 @@ export function dropDescendants (nodes) {
         undo: () => {
             for (let res of restore) {
                 res.node.children = res.children;
-                res.node._children = res._children;
             }
             dataUpdated();
         }