From 6c850f11f1811e4f3446baec9331fb487bb6c436 Mon Sep 17 00:00:00 2001
From: mvdbeek <m.vandenbeek@gmail.com>
Date: Fri, 9 Sep 2022 12:50:44 +0200
Subject: [PATCH] Add somewhat working Minimap

---
 .../components/Workflow/Editor/Draggable.vue  |   4 -
 .../src/components/Workflow/Editor/Index.vue  |   1 +
 .../components/Workflow/Editor/Minimap.vue    | 100 ++++++++++++++++++
 .../Workflow/Editor/MinimapNode.vue           |  18 ++++
 .../src/components/Workflow/Editor/Node.vue   |  11 +-
 .../components/Workflow/Editor/NodeInput.vue  |  10 +-
 .../components/Workflow/Editor/NodeOutput.vue |   8 +-
 .../Workflow/Editor/WorkflowGraph.vue         |  25 +++--
 client/src/store/workflowEditorStateStore.js  |  11 +-
 client/src/style/scss/workflow.scss           |   3 +
 10 files changed, 165 insertions(+), 26 deletions(-)
 create mode 100644 client/src/components/Workflow/Editor/Minimap.vue
 create mode 100644 client/src/components/Workflow/Editor/MinimapNode.vue

diff --git a/client/src/components/Workflow/Editor/Draggable.vue b/client/src/components/Workflow/Editor/Draggable.vue
index fe87aff9e0c..fff1b8df34d 100644
--- a/client/src/components/Workflow/Editor/Draggable.vue
+++ b/client/src/components/Workflow/Editor/Draggable.vue
@@ -19,10 +19,6 @@ export default {
         Draggable,
     },
     props: {
-        position: {
-            type: Object,
-            required: false,
-        },
         rootOffset: {
             type: Object,
             required: false,
diff --git a/client/src/components/Workflow/Editor/Index.vue b/client/src/components/Workflow/Editor/Index.vue
index 30d7149bcec..1c779431713 100644
--- a/client/src/components/Workflow/Editor/Index.vue
+++ b/client/src/components/Workflow/Editor/Index.vue
@@ -69,6 +69,7 @@
                     :steps="steps"
                     :datatypes-mapper="datatypesMapper"
                     :get-manager="getManager"
+                    :nodes="nodes"
                     @onUpdate="onUpdate"
                     @onClone="onClone"
                     @onCreate="onInsertTool"
diff --git a/client/src/components/Workflow/Editor/Minimap.vue b/client/src/components/Workflow/Editor/Minimap.vue
new file mode 100644
index 00000000000..87d3b7bf181
--- /dev/null
+++ b/client/src/components/Workflow/Editor/Minimap.vue
@@ -0,0 +1,100 @@
+<template>
+    <div class="workflow-overview" aria-hidden="true">
+        <div class="workflow-overview-body">
+            <div id="overview-container">
+                <svg width="100%" height="100%" :viewBox="viewBox">
+                    <MinimapNode
+                        class="mini-node"
+                        :node="node"
+                        :step="steps[node.id]"
+                        v-for="node of Object.values(nodes)"
+                        :key="node.id" />
+                    <rect
+                        class="mini-node"
+                        :x="visible.x"
+                        :y="visible.y"
+                        :width="visible.width"
+                        :height="visible.height"
+                        fill-opacity="0.5" />
+                </svg>
+                <div id="overview-viewport" />
+            </div>
+        </div>
+    </div>
+</template>
+<script>
+import MinimapNode from "./MinimapNode.vue";
+
+export default {
+    components: {
+        MinimapNode,
+    },
+    props: {
+        nodes: {
+            type: Object,
+            required: true,
+        },
+        steps: {
+            type: Object,
+            required: true,
+        },
+        width: {
+            type: Number,
+            default: 200,
+        },
+        height: {
+            type: Number,
+            default: 200,
+        },
+        scale: {
+            type: Number,
+            default: 1,
+        },
+        pan: {
+            type: Object,
+            default() {
+                return { x: 0, y: 0 };
+            },
+        },
+        rootOffset: {
+            type: Object,
+        },
+    },
+    computed: {
+        visible() {
+            // translate current rootOffset rectangle into scaled and panned space
+            const x = this.pan.x / -this.scale;
+            const y = this.pan.y / -this.scale;
+            const width = this.rootOffset.width / this.scale;
+            const height = this.rootOffset.bottom / this.scale;
+            const rval = { x, y, width, height };
+            return rval;
+        },
+        bounds() {
+            let left = this.pan.x * -1;
+            let right = this.rootOffset.width;
+            let top = this.pan.y * -1;
+            let bottom = this.rootOffset.bottom;
+            let p;
+            Object.values(this.steps).forEach((step) => {
+                const node = this.nodes[step.id];
+                p = node.position;
+                left = Math.min(left, step.position.left);
+                right = Math.max(right, p.right);
+                top = Math.min(top, step.position.top);
+                bottom = Math.max(bottom, p.bottom);
+            });
+
+            return {
+                left: left / this.scale,
+                right: right / this.scale,
+                top: top / this.scale,
+                bottom: bottom / this.scale,
+            };
+        },
+        viewBox() {
+            return `${this.bounds.left} ${this.bounds.top} ${this.bounds.right} ${this.bounds.bottom}`;
+        },
+    },
+};
+</script>
diff --git a/client/src/components/Workflow/Editor/MinimapNode.vue b/client/src/components/Workflow/Editor/MinimapNode.vue
new file mode 100644
index 00000000000..6dc70dd1f33
--- /dev/null
+++ b/client/src/components/Workflow/Editor/MinimapNode.vue
@@ -0,0 +1,18 @@
+<template>
+    <rect :x="step.position.left" :y="step.position.top" :width="node.position.width" :height="node.position.height" />
+</template>
+
+<script>
+export default {
+    props: {
+        node: {
+            type: Object,
+        },
+        step: {
+            type: Object,
+        },
+    },
+};
+</script>
+
+<style></style>
diff --git a/client/src/components/Workflow/Editor/Node.vue b/client/src/components/Workflow/Editor/Node.vue
index 6e3d4662de7..b1ce6d3c52d 100644
--- a/client/src/components/Workflow/Editor/Node.vue
+++ b/client/src/components/Workflow/Editor/Node.vue
@@ -1,6 +1,6 @@
 <template>
     <draggable-wrapper
-        :position="step.position || defaultPosition"
+        ref="el"
         @updatePosition="onUpdatePosition"
         @mouseup="makeActive"
         @move="onMove"
@@ -65,7 +65,6 @@
                 :key="input.name"
                 :input="input"
                 :get-node="getNode"
-                :get-manager="getManager"
                 :datatypes-mapper="datatypesMapper"
                 :root-offset="rootOffset"
                 :offset-x="offset.x"
@@ -81,7 +80,6 @@
                 :output="output"
                 :post-job-actions="postJobActions"
                 :get-node="getNode"
-                :get-manager="getManager"
                 :root-offset="rootOffset"
                 :offset-x="offset.x"
                 :offset-y="offset.y"
@@ -106,12 +104,19 @@ import NodeInput from "./NodeInput";
 import NodeOutput from "./NodeOutput";
 import DraggableWrapper from "./Draggable";
 import { ActiveOutputs } from "./modules/outputs";
+import { reactive, ref } from "vue";
+import { useElementBounding } from "@vueuse/core";
 
 Vue.use(BootstrapVue);
 
 const OFFSET_RANGE = 100;
 
 export default {
+    setup() {
+        const el = ref(null);
+        const position = reactive(useElementBounding(el));
+        return { el, position };
+    },
     components: {
         DraggableWrapper,
         LoadingSpan,
diff --git a/client/src/components/Workflow/Editor/NodeInput.vue b/client/src/components/Workflow/Editor/NodeInput.vue
index 5e60a0e962b..e2c7cd2f9b8 100644
--- a/client/src/components/Workflow/Editor/NodeInput.vue
+++ b/client/src/components/Workflow/Editor/NodeInput.vue
@@ -27,10 +27,6 @@ export default {
             type: Function,
             required: true,
         },
-        getManager: {
-            type: Function,
-            required: true,
-        },
         datatypesMapper: {
             type: Object,
             required: true,
@@ -126,5 +122,11 @@ export default {
             console.log("onDrop", e);
         },
     },
+    beforeDestroy() {
+        this.$store.commit("workflowState/deleteInputTerminalPosition", {
+            stepId: this.nodeId,
+            inputName: this.input.name,
+        });
+    },
 };
 </script>
diff --git a/client/src/components/Workflow/Editor/NodeOutput.vue b/client/src/components/Workflow/Editor/NodeOutput.vue
index c682042276a..59eb148bc5e 100644
--- a/client/src/components/Workflow/Editor/NodeOutput.vue
+++ b/client/src/components/Workflow/Editor/NodeOutput.vue
@@ -49,10 +49,6 @@ export default {
             type: Function,
             required: true,
         },
-        getManager: {
-            type: Function,
-            required: true,
-        },
         postJobActions: {
             type: Object,
             required: true,
@@ -170,6 +166,10 @@ export default {
         },
     },
     beforeDestroy() {
+        this.$store.commit("workflowState/deleteOutputTerminalPosition", {
+            stepId: this.getNode().id,
+            outputName: this.output.name,
+        });
         this.$emit("onRemove", this.output);
     },
     methods: {
diff --git a/client/src/components/Workflow/Editor/WorkflowGraph.vue b/client/src/components/Workflow/Editor/WorkflowGraph.vue
index c6f9bb6ee38..48dadbeb3ad 100644
--- a/client/src/components/Workflow/Editor/WorkflowGraph.vue
+++ b/client/src/components/Workflow/Editor/WorkflowGraph.vue
@@ -25,7 +25,7 @@
                             </g>
                         </svg>
                     </SvgPanZoom>
-                    <div class="nodeArea" ref="nodes" :style="style">
+                    <div class="nodeArea" :style="style">
                         <!-- this div is only necessary so that we synchronize zoom and pan to the svg coordinates -->
                         <WorkflowNode
                             v-for="(step, key) in steps"
@@ -49,14 +49,7 @@
                 </Draggable>
             </div>
         </div>
-        <div class="workflow-overview" aria-hidden="true">
-            <div class="workflow-overview-body">
-                <div id="overview-container">
-                    <canvas id="overview-canvas" width="0" height="0" />
-                    <div id="overview-viewport" />
-                </div>
-            </div>
-        </div>
+        <Minimap :nodes="nodes" :steps="steps" :root-offset="position" :scale="zoomLevel" :pan="currentPan" />
     </div>
 </template>
 <script>
@@ -65,6 +58,7 @@ import WorkflowNode from "./Node";
 import RawConnector from "./Connector";
 import TerminalConnector from "./TerminalConnector";
 import Draggable from "./Draggable.vue";
+import Minimap from "./Minimap.vue";
 import SvgPanZoom from "vue-svg-pan-zoom";
 import { reactive, ref } from "vue";
 import { useElementBounding } from "@vueuse/core";
@@ -81,6 +75,7 @@ export default {
         SvgPanZoom,
         TerminalConnector,
         WorkflowNode,
+        Minimap,
         ZoomControl,
     },
     data() {
@@ -104,6 +99,10 @@ export default {
             type: Object,
             default: null,
         },
+        nodes: {
+            type: Object,
+            required: true,
+        },
     },
     methods: {
         onUpdatedCTM(CTM) {
@@ -155,6 +154,11 @@ export default {
                 return `transform: matrix(${this.transform.a}, ${this.transform.b}, ${this.transform.c}, ${this.transform.d}, ${this.transform.e}, ${this.transform.f})`;
             }
         },
+        currentPan() {
+            if (this.transform) {
+                return { x: this.transform.e, y: this.transform.f };
+            }
+        },
         zoomLevel() {
             return this.$store.getters["workflowState/getScale"]();
         },
@@ -184,5 +188,8 @@ export default {
             return connections;
         },
     },
+    beforeDestroy() {
+        this.svgpanzoom.destroy();
+    },
 };
 </script>
diff --git a/client/src/store/workflowEditorStateStore.js b/client/src/store/workflowEditorStateStore.js
index 897a6d1ed37..bb761a14093 100644
--- a/client/src/store/workflowEditorStateStore.js
+++ b/client/src/store/workflowEditorStateStore.js
@@ -10,7 +10,6 @@ export const state = {
 import Vue from "vue";
 const getters = {
     getInputTerminalPosition: (state) => (stepId, inputName) => {
-        console.log("getting input for", stepId, inputName);
         return state.inputTerminals[stepId]?.[inputName];
     },
     getOutputTerminalPosition: (state) => (stepId, outputName) => {
@@ -47,6 +46,14 @@ const mutations = {
             Vue.set(state.outputTerminals[stepId], outputName, position);
         }
     },
+    deleteInputTerminalPosition: (state, { stepId, inputName }) => {
+        console.log("deleting output");
+        delete state.inputTerminals[stepId][inputName];
+    },
+    deleteOutputTerminalPosition: (state, { stepId, outputName }) => {
+        console.log("deleting output");
+        delete state.outputTerminals[stepId][outputName];
+    },
     setConnection: (state, { source, target, connection }) => {
         if (!state.connections[source]) {
             state.connections[source] = { target: connection };
@@ -61,7 +68,7 @@ const mutations = {
         state.scale = scale;
     },
     setNode: (state, node) => {
-        state.nodes[node.id] = node;
+        Vue.set(state.nodes, node.id, node);
     },
     deleteNode: (state, nodeId) => {
         delete state.nodes[nodeId];
diff --git a/client/src/style/scss/workflow.scss b/client/src/style/scss/workflow.scss
index 63381ddd4e1..6953929bca6 100644
--- a/client/src/style/scss/workflow.scss
+++ b/client/src/style/scss/workflow.scss
@@ -200,6 +200,9 @@
                     width: 100%;
                     height: 100%;
                 }
+                                .mini-node {
+                                    color: $brand-primary
+                                }
             }
             #input-choices-menu {
                 color: black;