From 11ac9125491c4d743c393857445b9b67ea5a437a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?H=C3=A9ctor=20Ram=C3=B3n=20Jim=C3=A9nez?= <hector@hecrj.dev>
Date: Wed, 18 Sep 2024 21:45:25 +0200
Subject: [PATCH] Fix `scrollable` transactions when `on_scroll` is not set

---
 widget/src/scrollable.rs | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/widget/src/scrollable.rs b/widget/src/scrollable.rs
index af6a3945fd..b8de66e32b 100644
--- a/widget/src/scrollable.rs
+++ b/widget/src/scrollable.rs
@@ -760,13 +760,17 @@ where
                     content_bounds,
                 );
 
-                if notify_scroll(
+                let has_scrolled = notify_scroll(
                     state,
                     &self.on_scroll,
                     bounds,
                     content_bounds,
                     shell,
-                ) {
+                );
+
+                let in_transaction = state.last_scrolled.is_some();
+
+                if has_scrolled || in_transaction {
                     event::Status::Captured
                 } else {
                     event::Status::Ignored
@@ -1194,11 +1198,6 @@ fn notify_viewport<Message>(
         return false;
     }
 
-    let Some(on_scroll) = on_scroll else {
-        state.last_notified = None;
-        return false;
-    };
-
     let viewport = Viewport {
         offset_x: state.offset_x,
         offset_y: state.offset_y,
@@ -1229,9 +1228,12 @@ fn notify_viewport<Message>(
         }
     }
 
-    shell.publish(on_scroll(viewport));
     state.last_notified = Some(viewport);
 
+    if let Some(on_scroll) = on_scroll {
+        shell.publish(on_scroll(viewport));
+    }
+
     true
 }