🐛 Fix support for renaming across tabs

yoavst · Dec 15, 2022 · d1bdde6 · d1bdde6
1 parent ddb77c7
commit d1bdde6
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/web/scripts/NetworkController.js b/web/scripts/NetworkController.js
@@ -9,13 +9,12 @@ class NetworkController {
             ws.send("MAGIC")
             document.getElementById("connectBtn").style.backgroundColor = "green"
         }
-        
+
         this.webSocket.onmessage = function (event) {
             const msg = JSON.parse(event.data);
 
-            tabsController.onCurrent((_, controller) => {
-
-                if (msg.type == MSG_ADD_NODE_AND_EDGE) {
+            if (msg.type == MSG_ADD_NODE_AND_EDGE) {
+                tabsController.onCurrent((_, controller) => {
                     const isNodeTarget = 'isNodeTarget' in msg ? msg.isNodeTarget : true
 
                     // 1. Find the selected node
@@ -41,12 +40,13 @@ class NetworkController {
                         // 4. selected added node
                         controller.selectNode(newNode.id)
                     }
-                } else if (msg.type == MSG_UPDATE_NODES) {
-                    controller.updateNodes(msg.selection, msg.update)
+                })
+            } else if (msg.type == MSG_UPDATE_NODES) {
+                // I assume all the opened tabs are from the same app, otherwise...
+                for (const { tabController } of tabsController.tabs) {
+                    tabController.updateNodes(msg.selection, msg.update)
                 }
-            })
-
-
+            }
             ws.send("MAGIC")
         }