From 28716248c5de9c9ebc9c8dbc1dd32ec7be9eb99b Mon Sep 17 00:00:00 2001
From: Jonathan Kelley <jkelleyrtp@gmail.com>
Date: Sat, 19 Feb 2022 17:34:44 -0500
Subject: [PATCH] fix: ping window after virtualdom is ready

---
 examples/heavy_compute.rs                  | 28 ++++++++++++++++++++++
 packages/desktop/src/controller.rs         |  3 +++
 packages/desktop/src/lib.rs                |  1 -
 packages/desktop/src/user_window_events.rs |  1 +
 4 files changed, 32 insertions(+), 1 deletion(-)
 create mode 100644 examples/heavy_compute.rs

diff --git a/examples/heavy_compute.rs b/examples/heavy_compute.rs
new file mode 100644
index 000000000..6e10a2f1f
--- /dev/null
+++ b/examples/heavy_compute.rs
@@ -0,0 +1,28 @@
+//! This example shows that you can place heavy work on the main thread, and then
+//!
+//! You *should* be using `tokio::spawn_blocking` instead.
+//!
+//! Your app runs in an async runtime (Tokio), so you should avoid blocking
+//! the rendering of the VirtualDom.
+//!
+//!
+
+use dioxus::prelude::*;
+
+fn main() {
+    dioxus::desktop::launch(app);
+}
+
+fn app(cx: Scope) -> Element {
+    // This is discouraged
+    std::thread::sleep(std::time::Duration::from_millis(2_000));
+
+    // This is suggested
+    tokio::task::spawn_blocking(move || {
+        std::thread::sleep(std::time::Duration::from_millis(2_000));
+    });
+
+    cx.render(rsx! {
+        div { "Hello, world!" }
+    })
+}
diff --git a/packages/desktop/src/controller.rs b/packages/desktop/src/controller.rs
index 8497a42e9..8715c74f0 100644
--- a/packages/desktop/src/controller.rs
+++ b/packages/desktop/src/controller.rs
@@ -57,6 +57,9 @@ impl DesktopController {
                     .unwrap()
                     .push_front(serde_json::to_string(&edits.edits).unwrap());
 
+                // Make sure the window is ready for any new updates
+                proxy.send_event(UserWindowEvent::Update).unwrap();
+
                 loop {
                     dom.wait_for_work().await;
                     let mut muts = dom.work_with_deadline(|| false);
diff --git a/packages/desktop/src/lib.rs b/packages/desktop/src/lib.rs
index 197735ddc..934c82f1a 100644
--- a/packages/desktop/src/lib.rs
+++ b/packages/desktop/src/lib.rs
@@ -143,7 +143,6 @@ pub fn launch_with_props<P: 'static + Send>(
                                         .send_event(user_window_events::UserWindowEvent::Update);
                                 }
                                 "browser_open" => {
-                                    println!("browser_open");
                                     let data = message.params();
                                     log::trace!("Open browser: {:?}", data);
                                     if let Some(temp) = data.as_object() {
diff --git a/packages/desktop/src/user_window_events.rs b/packages/desktop/src/user_window_events.rs
index aab14561b..bde215d48 100644
--- a/packages/desktop/src/user_window_events.rs
+++ b/packages/desktop/src/user_window_events.rs
@@ -3,6 +3,7 @@ use wry::application::window::Fullscreen as WryFullscreen;
 
 use crate::controller::DesktopController;
 
+#[derive(Debug)]
 pub(crate) enum UserWindowEvent {
     Update,