Set up the control path layer above not below, and starting collapsed

Author: Keavon

editor/src/messages/portfolio/document/document_message_handler.rs

@@ -2204,6 +2204,20 @@ impl DocumentMessageHandler {
 				responses.add(DocumentMessage::DocumentStructureChanged);
 				responses.add(NodeGraphMessage::SendGraph);
 
+				// The control path layer (Blend Path / Morph Path) should start collapsed.
+				let instance_path = {
+					// Build instance path from root down to the control path layer, which is a sibling of the main layer under `parent`.
+					let mut instance_path: Vec<NodeId> = parent
+						.ancestors(network_interface.document_metadata())
+						.take_while(|&ancestor| ancestor != LayerNodeIdentifier::ROOT_PARENT)
+						.map(LayerNodeIdentifier::to_node)
+						.collect();
+					instance_path.reverse();
+					instance_path.push(control_path_id);
+					instance_path
+				};
+				responses.add(DocumentMessage::ToggleLayerExpansion { instance_path, recursive: false });
+
 				return folder_id;
 			}
 		}

editor/src/messages/portfolio/document/graph_operation/graph_operation_message_handler.rs

@@ -193,8 +193,8 @@ impl MessageHandler<GraphOperationMessage, GraphOperationMessageContext<'_>> for
 				let control_path_layer = modify_inputs.create_layer(control_path_id);
 				let path_node_id = modify_inputs.insert_control_path_data(control_path_layer);
 
-				network_interface.move_layer_to_stack(layer, parent, insert_index, &[]);
-				network_interface.move_layer_to_stack(control_path_layer, parent, insert_index + 1, &[]);
+				network_interface.move_layer_to_stack(control_path_layer, parent, insert_index, &[]);
+				network_interface.move_layer_to_stack(layer, parent, insert_index + 1, &[]);
 
 				// Connect the Path node's output to the chain node's path parameter input (input 4 for both Morph and Blend).
 				// Done after move_layer_to_stack so chain nodes have correct positions when converted to absolute.