From c9873a0885bf50544bc2b72154dc257c66cb7959 Mon Sep 17 00:00:00 2001
From: Ivan Molodetskikh <yalterz@gmail.com>
Date: Tue, 12 Aug 2025 22:34:13 +0300
Subject: layout: Remove by-ref animation_snapshot() getter

It was used in only one place, and that place was due for an update to use the
cached data.
---
 src/layout/mod.rs       | 1 -
 src/layout/scrolling.rs | 8 +++-----
 src/layout/tests.rs     | 4 ----
 3 files changed, 3 insertions(+), 10 deletions(-)

(limited to 'src/layout')

diff --git a/src/layout/mod.rs b/src/layout/mod.rs
index 5deda707..7bf25b42 100644
--- a/src/layout/mod.rs
+++ b/src/layout/mod.rs
@@ -270,7 +270,6 @@ pub trait LayoutElement {
     /// Runs periodic clean-up tasks.
     fn refresh(&self);
 
-    fn animation_snapshot(&self) -> Option<&LayoutElementRenderSnapshot>;
     fn take_animation_snapshot(&mut self) -> Option<LayoutElementRenderSnapshot>;
 
     fn set_interactive_resize(&mut self, data: Option<InteractiveResizeData>);
diff --git a/src/layout/scrolling.rs b/src/layout/scrolling.rs
index 37feb227..94f2aa97 100644
--- a/src/layout/scrolling.rs
+++ b/src/layout/scrolling.rs
@@ -4041,15 +4041,13 @@ impl<W: LayoutElement> Column<W> {
             .find(|(_, tile)| tile.window().id() == window)
             .unwrap();
 
-        let height = f64::from(tile.window().size().h);
-        let offset = tile
-            .window()
-            .animation_snapshot()
-            .map_or(0., |from| from.size.h - height);
+        let prev_height = self.data[tile_idx].size.h;
 
         tile.update_window();
         self.data[tile_idx].update(tile);
 
+        let offset = prev_height - self.data[tile_idx].size.h;
+
         let is_tabbed = self.display_mode == ColumnDisplay::Tabbed;
 
         // Move windows below in tandem with resizing.
diff --git a/src/layout/tests.rs b/src/layout/tests.rs
index d7c66ba4..afb83d38 100644
--- a/src/layout/tests.rs
+++ b/src/layout/tests.rs
@@ -244,10 +244,6 @@ impl LayoutElement for TestWindow {
         &EMPTY
     }
 
-    fn animation_snapshot(&self) -> Option<&LayoutElementRenderSnapshot> {
-        None
-    }
-
     fn take_animation_snapshot(&mut self) -> Option<LayoutElementRenderSnapshot> {
         None
     }
-- 
cgit