From c8ef0e84e564508b9f1b8d7b35dee56c3dce82c3 Mon Sep 17 00:00:00 2001
From: Michael Dombrowski <mdombro@amazon.com>
Date: Thu, 26 Oct 2023 09:34:02 -0400
Subject: [PATCH] fix: enable pan/zoom in dependency graph

---
 src/main/js/dashboard-frontend/package.json   |  4 ++--
 .../components/details/DependencyGraph.tsx    | 21 +++++++++++++++----
 2 files changed, 19 insertions(+), 6 deletions(-)

diff --git a/src/main/js/dashboard-frontend/package.json b/src/main/js/dashboard-frontend/package.json
index 8b138ae..604f26d 100644
--- a/src/main/js/dashboard-frontend/package.json
+++ b/src/main/js/dashboard-frontend/package.json
@@ -4,8 +4,8 @@
   "private": true,
   "dependencies": {
     "@cloudscape-design/collection-hooks": "^1.0.23",
-    "@cloudscape-design/components": "^3.0.388",
     "@cloudscape-design/component-toolkit": "^1.0.0-beta.24",
+    "@cloudscape-design/components": "^3.0.388",
     "@cloudscape-design/design-tokens": "^3.0.28",
     "@cloudscape-design/global-styles": "^1.0.12",
     "ace-builds": "^1.4.12",
@@ -19,7 +19,7 @@
     "react-scripts": "^5.0.1",
     "react-tabs": "^3.2.3",
     "react-virtualized": "^9.21.2",
-    "sass": "^1.60.0",
+    "sass": "^1.69.5",
     "typescript": "^4.3.2",
     "use-persisted-state": "^0.3.3",
     "ws": "^7.4.6"
diff --git a/src/main/js/dashboard-frontend/src/components/details/DependencyGraph.tsx b/src/main/js/dashboard-frontend/src/components/details/DependencyGraph.tsx
index 024b5c3..2a417a1 100644
--- a/src/main/js/dashboard-frontend/src/components/details/DependencyGraph.tsx
+++ b/src/main/js/dashboard-frontend/src/components/details/DependencyGraph.tsx
@@ -126,11 +126,24 @@ class DependencyGraph extends Component<
   }
 
   drawGraph() {
-    let svg = d3.select(this.svg.current);
+    let svg = d3.select(this.svg.current!) as unknown as d3.Selection<Element, unknown, any, any>;
     let inner: any = d3.select(this.innerG.current);
+
     this.d3render(inner, this.g);
-    inner.attr("transform", `translate(20, 20)`);
-    svg.attr("height", this.g.graph().height + 40);
+
+    // Enable pan/zoom
+    const zoom = d3.zoom()
+        .on("zoom", () => {
+          inner.attr("transform", d3.event.transform);
+        });
+    svg.call(zoom);
+
+    const { width, height } = inner.node().getBBox();
+    // Scale to fit, but no bigger than 2x
+    const scale = Math.min(Math.min(this.svg.current!.clientWidth / width, this.svg.current!.clientHeight / height) * 0.95, 2);
+    zoom.scaleTo(svg, scale);
+    // Center horizontally
+    zoom.translateTo(svg, width / 2, 0);
   }
   async componentDidMount() {
     await SERVER.initConnections();
@@ -165,7 +178,7 @@ class DependencyGraph extends Component<
         disableContentPaddings={true}
       >
         <div className="holder">
-          <svg height="100" style={{width: "100%"}} ref={this.svg}>
+          <svg height="100" style={{width: "100%", height: "500px"}} ref={this.svg}>
             <g ref={this.innerG} />
           </svg>
         </div>