Node graph improvements (#855)

* Selecting multiple nodes

* Improve logs

* Log bad types in dyn any

* Add (broken) node links

* New topological sort

* Fix reorder ids function

* Input and output node

* Add nodes that operate on images

* Fixups

* Show node parameters together with layer properties

* New nodes don't crash editor

* Fix tests

* Node positions backend

* Generate node graph on value change

* Add expose input message

* Fix tests

Co-authored-by: Keavon Chambers <[email protected]>

Author: 0HyperCube

Cargo.lock

@@ -1,35 +1,35 @@
 use crate::messages::prelude::*;
 use graph_craft::document::{value::TaggedValue, NodeId};
-use graph_craft::proto::NodeIdentifier;
 
 #[remain::sorted]
 #[impl_message(Message, DocumentMessage, NodeGraph)]
 #[derive(PartialEq, Clone, Debug, serde::Serialize, serde::Deserialize)]
 pub enum NodeGraphMessage {
 	// Messages
-	AddLink {
-		from: NodeId,
-		to: NodeId,
-		to_index: usize,
-	},
 	CloseNodeGraph,
 	ConnectNodesByLink {
 		output_node: u64,
 		input_node: u64,
-		input_node_connector_index: u32,
+		input_node_connector_index: usize,
 	},
 	CreateNode {
 		// Having the caller generate the id means that we don't have to return it. This can be a random u64.
 		node_id: NodeId,
 		// I don't really know what this is for (perhaps a user identifiable name).
-		name: String,
-		// The node identifier must mach that found in `node-graph/graph-craft/src/node_registry.rs` e.g. "graphene_core::raster::GrayscaleNode
-		identifier: NodeIdentifier,
-		num_inputs: u32,
+		node_type: String,
 	},
 	DeleteNode {
 		node_id: NodeId,
 	},
+	ExposeInput {
+		node_id: NodeId,
+		input_index: usize,
+		new_exposed: bool,
+	},
+	MoveSelectedNodes {
+		displacement_x: i32,
+		displacement_y: i32,
+	},
 	OpenNodeGraph {
 		layer_path: Vec<graphene::LayerId>,
 	},