From 88b69cc593a1b8d265c133bbf71d4c8ce25dc4e3 Mon Sep 17 00:00:00 2001
From: Alon Levy <alon@pobox.com>
Date: Thu, 5 Feb 2015 12:54:50 +0200
Subject: client/graph_view: apply zoom of each node individually

Fixes #308

Hack for friday. Hack being that the model Node gets a 'zoom_obj'
property. This belongs on a new NodeView class that will contain a
reference to the node and the zoom object.

Also I'm worried about performance of this (well, just a little), have not tested with
something large yet.
---
 src/client/view/graph_view.js | 35 +++++++++++++++++++++++++++++++----
 1 file changed, 31 insertions(+), 4 deletions(-)

(limited to 'src')

diff --git a/src/client/view/graph_view.js b/src/client/view/graph_view.js
index 9e2a467a..c4ceefbd 100644
--- a/src/client/view/graph_view.js
+++ b/src/client/view/graph_view.js
@@ -333,6 +333,9 @@ function GraphView(spec) {
 
         var nodeEnter = node.enter()
             .append("g")
+            .each(function (d) {
+                d.zoom_obj = zoom_obj; // FIXME new object NodeView pointing to Node and Zoom
+             })
             .attr('id', function(d){ return d.id; }) // append node id to enable data->visual mapping
             .attr('visibility', 'hidden') // made visible on first tick
             .call(drag);
@@ -612,6 +615,20 @@ function GraphView(spec) {
         });
     }
 
+    /**
+     * Transform according to the parent transform
+     * @param d = {x:x, y:y}
+     * @return {x:new_x, y:new_y}
+     */
+    function apply_node_zoom_obj(d) {
+        var zoom_translate = d.zoom_obj.translate(),
+            zx = zoom_translate[0],
+            zy = zoom_translate[1],
+            s = d.zoom_obj.scale();
+
+        return [d.bx * s + zx, d.by * s + zy];
+    }
+
     function bubble_transform(d, bubble_radius) {
         if (bubble_radius == 0) {
             return d;
@@ -656,6 +673,14 @@ function GraphView(spec) {
         // transform nodes first to record bubble x & y (bx & by)
         node.attr("transform", transform);
 
+        function same_zoom(d) {
+            if (d.zoom_obj === null) {
+                return [d.bx, d.by];
+            } else {
+                return apply_node_zoom_obj(d);
+            }
+        }
+
         link.attr("d", function(d, i) {
             var d_val,
                 ghost;
@@ -663,9 +688,9 @@ function GraphView(spec) {
             util.assert(d.__src && d.__dst && d.__src.x && d.__src.y &&
                         d.__dst.x && d.__dst.y, "missing src and dst points");
 
-            var src = d.__src,
-                dst = d.__dst;
-            d_val = "M" + src.bx + "," + src.by + "L" + dst.bx + "," + dst.by;
+            var src = same_zoom(d.__src),
+                dst = same_zoom(d.__dst);
+            d_val = "M" + src[0] + "," + src[1] + "L" + dst[0] + "," + dst[1];
             // update ghostlink position
             ghost = $(this.nextElementSibling);
             ghost.attr("d", d_val);
@@ -673,7 +698,9 @@ function GraphView(spec) {
         });
 
         linktext.attr("transform", function(d) {
-            return "translate(" + (d.__src.bx + d.__dst.bx) / 2 + "," + (d.__src.by + d.__dst.by) / 2 + ")";
+            var src = same_zoom(d.__src),
+                dst = same_zoom(d.__dst);
+            return "translate(" + (src[0] + dst[0]) / 2 + "," + (src[1] + dst[1]) / 2 + ")";
         });
 
         // After initial placement we can make the nodes visible.
-- 
cgit v1.3.1