refactor(animation): force app rerenders

Author: thearturca

komorebi/src/window.rs

@@ -60,6 +60,7 @@ use std::time::Duration;
 use strum::Display;
 use strum::EnumString;
 use windows::Win32::Foundation::HWND;
+use windows::Win32::Graphics::Dwm::DwmFlush;
 
 pub static MINIMUM_WIDTH: AtomicI32 = AtomicI32::new(0);
 pub static MINIMUM_HEIGHT: AtomicI32 = AtomicI32::new(0);
@@ -197,6 +198,8 @@ impl RenderDispatcher for MovementRenderDispatcher {
         stackbar_manager::STACKBAR_TEMPORARILY_DISABLED.store(true, Ordering::SeqCst);
         stackbar_manager::send_notification();
 
+        WindowsApi::send_enter_size_move(self.hwnd)?;
+
         Ok(())
     }
 
@@ -205,16 +208,22 @@ impl RenderDispatcher for MovementRenderDispatcher {
 
         // we don't check WINDOW_HANDLING_BEHAVIOUR here because animations
         // are always run on a separate thread
-        WindowsApi::move_window(self.hwnd, &new_rect, false)?;
-        WindowsApi::invalidate_rect(self.hwnd, None, false);
+        WindowsApi::move_window(self.hwnd, &new_rect, true)?;
+
+        // force window redraw
+        WindowsApi::invalidate_rect(self.hwnd, None, true);
+        WindowsApi::update_window(self.hwnd);
 
+        // wait for dwm to flush render queue
+        unsafe { DwmFlush() }?;
         Ok(())
     }
 
     fn post_render(&self) -> Result<()> {
         // we don't add the async_window_pos flag here because animations
         // are always run on a separate thread
         WindowsApi::position_window(self.hwnd, &self.target_rect, self.top, false)?;
+        WindowsApi::send_exit_size_move(self.hwnd)?;
         if ANIMATION_MANAGER
             .lock()
             .count_in_progress(MovementRenderDispatcher::PREFIX)

komorebi/src/windows_api.rs

@@ -41,6 +41,7 @@ use windows::Win32::Graphics::Gdi::MonitorFromPoint;
 use windows::Win32::Graphics::Gdi::MonitorFromWindow;
 use windows::Win32::Graphics::Gdi::Rectangle;
 use windows::Win32::Graphics::Gdi::RoundRect;
+use windows::Win32::Graphics::Gdi::UpdateWindow;
 use windows::Win32::Graphics::Gdi::HBRUSH;
 use windows::Win32::Graphics::Gdi::HDC;
 use windows::Win32::Graphics::Gdi::HMONITOR;
@@ -100,6 +101,7 @@ use windows::Win32::UI::WindowsAndMessaging::PostMessageW;
 use windows::Win32::UI::WindowsAndMessaging::RealGetWindowClassW;
 use windows::Win32::UI::WindowsAndMessaging::RegisterClassW;
 use windows::Win32::UI::WindowsAndMessaging::RegisterDeviceNotificationW;
+use windows::Win32::UI::WindowsAndMessaging::SendMessageW;
 use windows::Win32::UI::WindowsAndMessaging::SetCursorPos;
 use windows::Win32::UI::WindowsAndMessaging::SetForegroundWindow;
 use windows::Win32::UI::WindowsAndMessaging::SetLayeredWindowAttributes;
@@ -139,6 +141,8 @@ use windows::Win32::UI::WindowsAndMessaging::SYSTEM_PARAMETERS_INFO_ACTION;
 use windows::Win32::UI::WindowsAndMessaging::SYSTEM_PARAMETERS_INFO_UPDATE_FLAGS;
 use windows::Win32::UI::WindowsAndMessaging::WINDOW_LONG_PTR_INDEX;
 use windows::Win32::UI::WindowsAndMessaging::WM_CLOSE;
+use windows::Win32::UI::WindowsAndMessaging::WM_ENTERSIZEMOVE;
+use windows::Win32::UI::WindowsAndMessaging::WM_EXITSIZEMOVE;
 use windows::Win32::UI::WindowsAndMessaging::WNDCLASSW;
 use windows::Win32::UI::WindowsAndMessaging::WNDENUMPROC;
 use windows::Win32::UI::WindowsAndMessaging::WS_DISABLED;
@@ -1360,6 +1364,24 @@ impl WindowsApi {
         unsafe { InvalidateRect(Option::from(HWND(as_ptr!(hwnd))), rect, erase) }.as_bool()
     }
 
+    pub fn send_enter_size_move(hwnd: isize) -> Result<()> {
+        unsafe {
+            SendMessageW(HWND(as_ptr!(hwnd)), WM_ENTERSIZEMOVE, None, None);
+            Ok(())
+        }
+    }
+
+    pub fn send_exit_size_move(hwnd: isize) -> Result<()> {
+        unsafe {
+            SendMessageW(HWND(as_ptr!(hwnd)), WM_EXITSIZEMOVE, None, None);
+            Ok(())
+        }
+    }
+
+    pub fn update_window(hwnd: isize) -> bool {
+        unsafe { UpdateWindow(HWND(as_ptr!(hwnd))) }.as_bool()
+    }
+
     pub fn alt_is_pressed() -> bool {
         let state = unsafe { GetKeyState(i32::from(VK_MENU.0)) };
         #[allow(clippy::cast_sign_loss)]