Desktop: Focus app window when loading a document (#4124)

desktop/src/app.rs

@@ -368,6 +368,11 @@ impl App {
 					window.start_drag();
 				}
 			}
+			DesktopFrontendMessage::WindowFocus => {
+				if let Some(window) = &self.window {
+					window.focus();
+				}
+			}
 			DesktopFrontendMessage::WindowHide => {
 				if let Some(window) = &self.window {
 					window.hide();

desktop/src/window.rs

@@ -140,6 +140,11 @@ impl Window {
 		let _ = self.winit_window.drag_window();
 	}
 
+	pub(crate) fn focus(&self) {
+		self.winit_window.set_minimized(false);
+		self.winit_window.focus_window();
+	}
+
 	pub(crate) fn hide(&self) {
 		self.native_handle.hide();
 	}

desktop/wrapper/src/intercept_frontend_message.rs

@@ -139,6 +139,9 @@ pub(super) fn intercept_frontend_message(dispatcher: &mut DesktopWrapperMessageD
 		FrontendMessage::WindowDrag => {
 			dispatcher.respond(DesktopFrontendMessage::WindowDrag);
 		}
+		FrontendMessage::WindowFocus => {
+			dispatcher.respond(DesktopFrontendMessage::WindowFocus);
+		}
 		FrontendMessage::WindowHide => {
 			dispatcher.respond(DesktopFrontendMessage::WindowHide);
 		}

desktop/wrapper/src/messages.rs

@@ -72,6 +72,7 @@ pub enum DesktopFrontendMessage {
 	WindowMaximize,
 	WindowFullscreen,
 	WindowDrag,
+	WindowFocus,
 	WindowHide,
 	WindowHideOthers,
 	WindowShowAll,

editor/src/messages/app_window/app_window_message.rs

@@ -11,6 +11,7 @@ pub enum AppWindowMessage {
 	Maximize,
 	Fullscreen,
 	Drag,
+	Focus,
 	Hide,
 	HideOthers,
 	ShowAll,

editor/src/messages/app_window/app_window_message_handler.rs

@@ -36,6 +36,10 @@ impl MessageHandler<AppWindowMessage, ()> for AppWindowMessageHandler {
 				#[cfg(not(target_family = "wasm"))]
 				responses.add(FrontendMessage::WindowDrag);
 			}
+			AppWindowMessage::Focus => {
+				#[cfg(not(target_family = "wasm"))]
+				responses.add(FrontendMessage::WindowFocus);
+			}
 			AppWindowMessage::Hide => {
 				#[cfg(not(target_family = "wasm"))]
 				responses.add(FrontendMessage::WindowHide);

editor/src/messages/frontend/frontend_message.rs

@@ -362,6 +362,8 @@ pub enum FrontendMessage {
 	#[cfg(not(target_family = "wasm"))]
 	WindowHide,
 	#[cfg(not(target_family = "wasm"))]
+	WindowFocus,
+	#[cfg(not(target_family = "wasm"))]
 	WindowHideOthers,
 	#[cfg(not(target_family = "wasm"))]
 	WindowShowAll,

editor/src/messages/portfolio/portfolio_message_handler.rs

@@ -819,6 +819,8 @@ impl MessageHandler<PortfolioMessage, PortfolioMessageContext<'_>> for Portfolio
 
 				// Load the document into the portfolio so it opens in the editor
 				self.load_document(document, document_id, responses);
+
+				responses.add(AppWindowMessage::Focus);
 			}
 			PortfolioMessage::OpenImage { name, image } => {
 				// `NewDocumentWithName`'s handler routes empty/None-equivalent names through `resolve_document_name` which assigns the next available "Untitled Document {N}".