fix: keep offscreen kitty images loaded instead of resending them on scroll (#4686)

1 files changed, 10 insertions, 6 deletions

diff --git a/src/kitty_graphics.rs b/src/kitty_graphics.rs
index e69b9df3..f26cd8d9 100644
--- a/src/kitty_graphics.rs
+++ b/src/kitty_graphics.rs
@@ -335,16 +335,20 @@ fn collect_visible_placements(
             }
         };
         let mut requested_images = HashSet::new();
-        for placement in runtime.kitty_image_placements_with_data_filter(|descriptor| {
+        let pane_placements = runtime.kitty_image_placements_with_data_filter(|descriptor| {
             if descriptor.source_file {
                 return false;
             }
             terminal_image_needs_data(info.id, descriptor, delivered_images, &mut requested_images)
-        }) {
-            let scrollback_offset = runtime
-                .scroll_metrics()
-                .map(|m| m.offset_from_bottom as u32)
-                .unwrap_or(0);
+        });
+        if pane_placements.is_empty() {
+            continue;
+        }
+        let scrollback_offset = runtime
+            .scroll_metrics()
+            .map(|m| m.offset_from_bottom as u32)
+            .unwrap_or(0);
+        for placement in pane_placements {
             placements.push(HostPlacement {
                 raw_data: None,
                 pane_id: info.id,