Combine group moving to a single transaction, refs #59

flowhub · Feb 21, 2014 · 035c730 · forresto · Feb 21, 2014 · 035c730
1 parent 607d387
commit 035c730
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/the-graph/the-graph-graph.js b/the-graph/the-graph-graph.js
@@ -92,6 +92,8 @@
       var graph = this.state.graph;
       var nodes = event.detail.nodes;
 
+      graph.startTransaction('movegroup');
+
       // Move each group member
       var len = nodes.length;
       for (var i=0; i<len; i++) {
@@ -103,6 +105,8 @@
           });
         }
       }
+
+      graph.endTransaction('movegroup');
     },
     listenComponentChanges: function (componentName) {
       var componentEl = document.querySelector("the-component[name='"+componentName+"']");

diff --git a/the-graph/the-graph-node.js b/the-graph/the-graph-node.js
@@ -32,6 +32,8 @@
 
       this.getDOMNode().addEventListener("track", this.onTrack);
       this.getDOMNode().addEventListener("trackend", this.onTrackEnd);
+
+      //this.props.graph.startTransaction('movenode');
     },
     onTrack: function (event) {
       // Don't fire on graph
@@ -65,6 +67,7 @@
 
       this.getDOMNode().removeEventListener("track", this.onTrack);
       this.getDOMNode().removeEventListener("trackend", this.onTrackEnd);
+      //this.props.graph.endTransaction('movenode');
     },
     stopPropagation: function (event) {
       // HACK to keep context menu from cancelling preview edge
@@ -316,4 +319,4 @@
   });
 
 
-})(this);
+})(this);