From b56bd7bb5ea82b80ae87c3004ef347e8750b1873 Mon Sep 17 00:00:00 2001 From: Nick O'Leary Date: Mon, 20 Mar 2023 17:15:45 +0000 Subject: [PATCH] Fix group selection when using lasso --- .../@node-red/editor-client/src/js/ui/view.js | 17 ++++++++++++----- 1 file changed, 12 insertions(+), 5 deletions(-) diff --git a/packages/node_modules/@node-red/editor-client/src/js/ui/view.js b/packages/node_modules/@node-red/editor-client/src/js/ui/view.js index dbde88a9a7..5caa3d98f7 100644 --- a/packages/node_modules/@node-red/editor-client/src/js/ui/view.js +++ b/packages/node_modules/@node-red/editor-client/src/js/ui/view.js @@ -189,7 +189,13 @@ RED.view = (function() { set.unshift(...removed) } }, - find: function(func) { return set.find(func) } + find: function(func) { return set.find(func) }, + dump: function () { + console.log('MovingSet Contents') + api.forEach((n, i) => { + console.log(`${i+1}\t${n.n.id}\t${n.n.type}`) + }) + } } return api; })(); @@ -1920,10 +1926,11 @@ RED.view = (function() { if (!movingSet.has(n) && !n.selected) { // group entirely within lasso if (n.x > x && n.y > y && n.x + n.w < x2 && n.y + n.h < y2) { - n.selected = true - n.dirty = true - var groupNodes = RED.group.getNodes(n,true); - groupNodes.forEach(gn => movingSet.add(gn)) + selectGroup(n, true) + // n.selected = true + // n.dirty = true + // var groupNodes = RED.group.getNodes(n,true); + // groupNodes.forEach(gn => movingSet.add(gn)) } } })