From 8a0d540552cda0c697b97dcd2f5d4b48bf2ff376 Mon Sep 17 00:00:00 2001 From: Philipp Oeser Date: Fri, 2 Jun 2023 11:14:52 +0200 Subject: [PATCH 1/2] Fix: Nodes pasting can leave multiple active nodes Clear active flag on pasted nodes (this is in line with how objects are pasted). Could alternatively be handled in node_copy_with_mapping? Part of #108049 --- source/blender/editors/space_node/clipboard.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/source/blender/editors/space_node/clipboard.cc b/source/blender/editors/space_node/clipboard.cc index 5ed91cd4fbe..2f7da1dfc78 100644 --- a/source/blender/editors/space_node/clipboard.cc +++ b/source/blender/editors/space_node/clipboard.cc @@ -269,6 +269,9 @@ static int node_clipboard_paste_exec(bContext *C, wmOperator *op) for (bNode *new_node : node_map.values()) { nodeSetSelected(new_node, true); + /* Keep previous active node, this is in line with how objects are pasted. */ + new_node->flag &= ~NODE_ACTIVE; + /* The parent pointer must be redirected to new node. */ if (new_node->parent) { if (node_map.contains(new_node->parent)) { -- 2.30.2 From 439e9ea503fa879cb054357cbb61c14e52770d8a Mon Sep 17 00:00:00 2001 From: Philipp Oeser Date: Sun, 18 Jun 2023 12:08:57 +0200 Subject: [PATCH 2/2] remove comment --- source/blender/editors/space_node/clipboard.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/source/blender/editors/space_node/clipboard.cc b/source/blender/editors/space_node/clipboard.cc index 2f7da1dfc78..9d587878fe4 100644 --- a/source/blender/editors/space_node/clipboard.cc +++ b/source/blender/editors/space_node/clipboard.cc @@ -269,7 +269,6 @@ static int node_clipboard_paste_exec(bContext *C, wmOperator *op) for (bNode *new_node : node_map.values()) { nodeSetSelected(new_node, true); - /* Keep previous active node, this is in line with how objects are pasted. */ new_node->flag &= ~NODE_ACTIVE; /* The parent pointer must be redirected to new node. */ -- 2.30.2