fix: prevent moving tiling windows with mouse from affecting hidden w…

…orkspaces (#877)
glzr-io · Dec 2, 2024 · ac4ae08 · ac4ae08
1 parent ee9f756
commit ac4ae08
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/wm/src/common/events/handle_window_moved_or_resized_end.rs b/packages/wm/src/common/events/handle_window_moved_or_resized_end.rs
@@ -107,7 +107,7 @@ fn drop_as_tiling_window(
   // Get the workspace, split containers, and other windows under the
   // dragged window.
   let containers_at_pos = state
-    .containers_at_point(&mouse_pos)
+    .containers_at_point(workspace.clone().into(), &mouse_pos)
     .into_iter()
     .filter(|container| container.id() != moved_window.id());
 

diff --git a/packages/wm/src/wm_state.rs b/packages/wm/src/wm_state.rs
@@ -502,9 +502,12 @@ impl WmState {
       .or(Some(workspace.into()))
   }
 
-  pub fn containers_at_point(&self, point: &Point) -> Vec<Container> {
-    self
-      .root_container
+  pub fn containers_at_point(
+    &self,
+    origin_container: Container,
+    point: &Point,
+  ) -> Vec<Container> {
+    origin_container
       .descendants()
       .filter(|descendant| {
         descendant