Cleanup

Author: TrueDoctor

editor/src/messages/defer/defer_message_handler.rs

@@ -17,7 +17,6 @@ impl MessageHandler<DeferMessage, DeferMessageContext<'_>> for DeferMessageHandl
 	fn process_message(&mut self, message: DeferMessage, responses: &mut VecDeque<Message>, context: DeferMessageContext) {
 		match message {
 			DeferMessage::AfterGraphRun { mut messages } => {
-				log::debug!("queing run");
 				let after_graph_run = self.after_graph_run.entry(context.portfolio.active_document_id.unwrap_or(DocumentId(0))).or_default();
 				after_graph_run.extend(messages.drain(..).map(|m| (self.current_graph_submission_id, m)));
 				responses.add(NodeGraphMessage::RunDocumentGraph);
@@ -29,7 +28,6 @@ impl MessageHandler<DeferMessage, DeferMessageContext<'_>> for DeferMessageHandl
 				self.current_graph_submission_id = execution_id + 1;
 			}
 			DeferMessage::TriggerGraphRun(execution_id, document_id) => {
-				log::debug!("trigger run");
 				let after_graph_run = self.after_graph_run.entry(document_id).or_default();
 				if after_graph_run.is_empty() {
 					return;
@@ -47,7 +45,6 @@ impl MessageHandler<DeferMessage, DeferMessageContext<'_>> for DeferMessageHandl
 				}
 			}
 			DeferMessage::TriggerNavigationReady => {
-				log::debug!("navigation ready");
 				for message in self.after_viewport_resize.drain(..).rev() {
 					responses.add_front(message);
 				}

editor/src/messages/dialog/new_document_dialog/new_document_dialog_message_handler.rs

@@ -33,32 +33,16 @@ impl<'a> MessageHandler<NewDocumentDialogMessage, NewDocumentDialogMessageContex
 						id: NodeId::new(),
 						artboard: graphene_std::Artboard::new(IVec2::ZERO, self.dimensions.as_ivec2()),
 					});
-					responses.add(NavigationMessage::CanvasPan {
-						delta: self.dimensions.as_dvec2() * -0.5,
-					});
+					responses.add(NavigationMessage::CanvasPan { delta: self.dimensions.as_dvec2() });
 					responses.add(NodeGraphMessage::RunDocumentGraph);
+					// If we already have bounds, we won't receive a viewport bounds update so we just fabricate one ourselves
 					if *context.viewport_bounds != ViewportBounds::default() {
-						log::debug!("already got bounds");
 						responses.add(InputPreprocessorMessage::BoundsOfViewports {
 							bounds_of_viewports: vec![context.viewport_bounds.clone()],
 						});
 					}
 					responses.add(DeferMessage::AfterNavigationReady {
-						messages: vec![
-							// DeferMessage::AfterGraphRun {
-							// 	messages: vec![
-							// 		DeferMessage::AfterGraphRun {
-							// 			messages: vec![DocumentMessage::ZoomCanvasToFitAll.into(), DocumentMessage::DeselectAllLayers.into()],
-							// 		}
-							// 		.into(),
-							// 		DocumentMessage::ZoomCanvasToFitAll.into(),
-							// 		DocumentMessage::DeselectAllLayers.into(),
-							// 	],
-							// }
-							// .into(),
-							DocumentMessage::ZoomCanvasToFitAll.into(),
-							DocumentMessage::DeselectAllLayers.into(),
-						],
+						messages: vec![DocumentMessage::ZoomCanvasToFitAll.into(), DocumentMessage::DeselectAllLayers.into()],
 					});
 				}
 			}