From f34079c86550f4a286ec9fe8c3abd456cb9e5bec Mon Sep 17 00:00:00 2001
From: CJ Dimaano <c.j.s.dimaano@gmail.com>
Date: Wed, 24 May 2017 10:34:21 -0700
Subject: [PATCH] address #222

---
 app/components/graph-editor/editor-graph.ts | 38 ++++++++++-----------
 1 file changed, 18 insertions(+), 20 deletions(-)

diff --git a/app/components/graph-editor/editor-graph.ts b/app/components/graph-editor/editor-graph.ts
index 3e43da8..031cdf6 100644
--- a/app/components/graph-editor/editor-graph.ts
+++ b/app/components/graph-editor/editor-graph.ts
@@ -10,11 +10,6 @@
  *   (except for events).
  */
 
-// TODO: Make sure to not rely on this dependency long term.
-// import { PLUGIN_DIRECTORY } from "../../services/plugin.service";
-// import { remote } from "electron";
-// const path = remote.require("path");
-
 import { NUDGE, SELECTION_COLOR, STICKY_DELAY } from "./defaults";
 import {
     DrawableEvent,
@@ -268,6 +263,9 @@ export class EditorGraph {
 
             // Update the drag point.
             this.dragPoint = pt;
+
+            // Redraw the canvas.
+            this.draw();
         }
     }
 
@@ -328,6 +326,7 @@ export class EditorGraph {
      *   True if the hover object is set to anything; otherwise, false.
      */
     updateHoverObject(pt?: point): boolean {
+        this.suspendDraw();
 
         // Update the previous hover object.
         if (this.hoverObject) {
@@ -437,7 +436,7 @@ export class EditorGraph {
         else
             this.hoverObject = null;
 
-        this.draw();
+        this.resumeDraw();
         return this.hoverObject !== null;
     }
 
@@ -471,7 +470,6 @@ export class EditorGraph {
                     this.dragObject.isDragging = true;
                 }
             }
-            this.draw();
         }
 
         // Pick up the edge if one is being hovered.
@@ -481,6 +479,7 @@ export class EditorGraph {
             const apt = (this.drawables
                 .get(hoverEdge.drawable.source) as EditorNode).anchor;
             const isSrc = spt.x !== apt.x || spt.y !== apt.y;
+            this.updateHoverObject();
             const edge = this.createDragEdge(
                 (isSrc ? hoverEdge.source : hoverEdge.destination),
                 isSrc,
@@ -498,9 +497,8 @@ export class EditorGraph {
                 else
                     edge.bindDestinationAnchor(pt);
             }
-            this.hoverObject.isDragging = true;
-            this.moveEdge = this.hoverObject;
-            this.updateHoverObject();
+            hoverEdge.isDragging = true;
+            this.moveEdge = hoverEdge;
         }
 
         // Clear the drag object if the hover object is not a node or edge.
@@ -508,7 +506,6 @@ export class EditorGraph {
             this.dragObject.isDragging = false;
             this.dragObject = null;
             this.updateHoverObject();
-            this.draw();
         }
     }
 
@@ -541,9 +538,6 @@ export class EditorGraph {
         this._drawSelectionBox = () => {
             this.g.drawSelectionBox(rect);
         };
-
-        // Update the canvas.
-        this.draw();
     }
 
     /**
@@ -563,7 +557,7 @@ export class EditorGraph {
      *   Updates the collection of nodes being dragged.
      */
     private updateDragNodes(dragNode: EditorNode, dpt: point) {
-        this.suspendDraw();
+        // this.suspendDraw();
         const selectedNodes = [...this.drawable.selectedNodes];
         if (dragNode.drawable.isSelected && selectedNodes.length > 0) {
             for (const n of selectedNodes)
@@ -574,7 +568,7 @@ export class EditorGraph {
         }
         else
             this.updateDragNode(dragNode, dpt);
-        this.resumeDraw();
+        // this.resumeDraw();
     }
 
     /**
@@ -682,7 +676,7 @@ export class EditorGraph {
         this.moveEdge = null;
         // Move or create the edge if it was dropped on a node.
         if (hoverNode instanceof EditorNode) {
-            this.suspendDraw();
+            // this.suspendDraw();
             // Get the source and destination nodes.
             const src = dragEdge.source === HIDDEN_NODE ?
                 hoverNode :
@@ -721,7 +715,7 @@ export class EditorGraph {
                     );
                 this.updateSelected(drawable);
             }
-            this.resumeDraw();
+            // this.resumeDraw();
         }
         // Update the original edge if one was being moved.
         else if (like) {
@@ -795,7 +789,6 @@ export class EditorGraph {
             this.unselected.add(e);
         e.update(this.g);
         d.addEventListener("change", this.onDrawablePropertyChanged);
-        this.draw();
     }
 
     /**
@@ -821,7 +814,6 @@ export class EditorGraph {
         this.unselected.delete(e);
         this.drawables.delete(d);
         d.removeEventListener("change", this.onDrawablePropertyChanged);
-        this.draw();
     }
 
     /**
@@ -869,8 +861,10 @@ export class EditorGraph {
      */
     private onCreated
     = (evt: DrawableEvent<DrawableElement>) => {
+        this.suspendDraw();
         for (const e of evt.detail.drawables)
             this.registerDrawable(e[0]);
+        this.resumeDraw();
     }
 
     /**
@@ -881,9 +875,11 @@ export class EditorGraph {
      */
     private onDeleted
     = (evt: DrawableEvent<DrawableElement>) => {
+        this.suspendDraw();
         for (const e of evt.detail.drawables) {
             this.unregisterDrawable(e[0]);
         }
+        this.resumeDraw();
     }
 
     /**
@@ -893,8 +889,10 @@ export class EditorGraph {
      */
     private onMovedEdge
     = (evt: MoveEdgeEvent) => {
+        this.suspendDraw();
         this.unregisterDrawable(evt.detail.original);
         this.registerDrawable(evt.detail.replacement);
+        this.resumeDraw();
     }
 
     /**