From 1f8915f80210635cacc4b8d222f54b0b8c26c85f Mon Sep 17 00:00:00 2001
From: Victoria Brekenfeld <victoria@system76.com>
Date: Thu, 8 Aug 2024 20:48:26 +0200
Subject: [PATCH] layout/tiling: Don't unmap before moving children

---
 src/shell/layout/tiling/mod.rs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/shell/layout/tiling/mod.rs b/src/shell/layout/tiling/mod.rs
index 595a9ebe..8607fb56 100644
--- a/src/shell/layout/tiling/mod.rs
+++ b/src/shell/layout/tiling/mod.rs
@@ -689,7 +689,6 @@ impl TilingLayout {
                     .map(|child_id| (desc.node.clone(), child_id.clone()))
                     .collect::<Vec<_>>();
                 let node = Node::new(node.data().clone());
-                TilingLayout::unmap_internal(&mut this_tree, &desc.node);
 
                 let id = match other_tree.root_node_id() {
                     None => other_tree.insert(node, InsertBehavior::AsRoot).unwrap(),
@@ -786,6 +785,7 @@ impl TilingLayout {
                 let this_gaps = this.gaps();
                 let other_gaps = other.gaps();
 
+                TilingLayout::unmap_internal(&mut this_tree, &desc.node);
                 let blocker =
                     TilingLayout::update_positions(&this.output, &mut this_tree, this_gaps);
                 this.queue.push_tree(this_tree, ANIMATION_DURATION, blocker);