Browse Source

Merge pull request #87125 from ryevdokimov/fix-removing-preview-nodes-on-ui-cancel

Clear drag preview nodes on `NOTIFICATION_DRAG_END`
Rémi Verschelde 1 year ago
parent
commit
e096be85c4

+ 4 - 0
editor/plugins/canvas_item_editor_plugin.cpp

@@ -6030,6 +6030,10 @@ void CanvasItemEditorViewport::_notification(int p_what) {
 		case NOTIFICATION_EXIT_TREE: {
 			disconnect("mouse_exited", callable_mp(this, &CanvasItemEditorViewport::_on_mouse_exit));
 		} break;
+
+		case NOTIFICATION_DRAG_END: {
+			_remove_preview();
+		} break;
 	}
 }
 

+ 2 - 0
editor/plugins/node_3d_editor_plugin.cpp

@@ -3017,6 +3017,8 @@ void Node3DEditorViewport::_notification(int p_what) {
 			// Clear preview material when dropped outside applicable object.
 			if (spatial_editor->get_preview_material().is_valid() && !is_drag_successful()) {
 				_remove_preview_material();
+			} else {
+				_remove_preview_node();
 			}
 		} break;
 	}