Liść na wietrze

qarmin · qarmin · commit 20c0181666b7 · 2026-04-07T20:53:47.000+02:00
diff --git a/cedinia/AGENTS.md b/cedinia/AGENTS.md
@@ -69,16 +69,48 @@ cedinia/src/
     ├── colors.slint               # CediniaColors theme
     ├── settings_screen.slint      # Settings screen layout
     ├── settings_components.slint  # ToggleRow, SegmentRow, DropdownRow, TextInputRow
-    ├── results_list.slint         # Scrollable results list (MomentumScrollView)
     ├── similar_images_gallery.slint
     ├── directories_screen.slint   # Add/remove directory paths
-    ├── momentum_scroll.slint      # Touch momentum scroll (custom component)
     ├── components.slint           # Shared components
     └── …
 ```
 
 ---
 
+## Slint Performance: ListView vs. for-in / ScrollView
+
+When displaying more than a few dozen items, **always use `ListView`** (from
+`std-widgets.slint`)
+rather than a bare `ScrollView` / `for` loop inside a `VerticalLayout`.
+
+```slint
+// SLOW – instantiates every item upfront, O(n) per frame
+ScrollView {
+    VerticalLayout {
+        for item in model : Row { ... }
+    }
+}
+
+// SLOW - same as above
+VerticalLayout {
+    for item in model : Row { ... }
+}
+
+// FAST – virtual scroll via Flickable's Repeater optimization
+ListView {
+    for item in model : Row { ... }
+}
+```
+
+`ListView` uses Slint's Repeater-inside-Flickable
+optimization: only visible rows (plus a small buffer) are instantiated at any
+given time.  With a plain `ScrollView` + `VerticalLayout`, all N items are
+created and visited on every frame.
+
+Reference: Slint issue [#11021](https://github.com/slint-ui/slint/issues/11021).
+
+---
+
 ## Scan Data Flow
 
 ```
@@ -112,13 +144,6 @@ Custom settings row with a scrollable popup:
 - Content wrapped in `ScrollView` with `viewport-height = options.length * 48px`,
   allowing the 27-language list to scroll.
 
-### MomentumScrollView (`ui/momentum_scroll.slint`)
-
-Custom touch-scroll component used for the results list and image gallery.
-Provides momentum / deceleration on swipe. Not available in krokiet.
-
----
-
 ## Translation System
 
 Same workaround as krokiet:
@@ -188,7 +213,6 @@ and used by `czkawka_core`'s config path logic.
 | Video tools | No | Yes |
 | Video optimizer | No | Yes |
 | Bad names tool | Yes | No |
-| Touch scroll | MomentumScrollView | Standard |
 | File picker | JNI (Android) / rfd (desktop) | rfd |
 | Settings | JSON, no presets | JSON, 11 presets |
 | Column indices | `common.rs` (own set) | `common.rs` (own set) |
diff --git a/cedinia/src/app.rs b/cedinia/src/app.rs
@@ -238,8 +238,6 @@ impl ScanResultHandler for GuiHandler {
                     win.set_similar_images_model(make_file_model(items));
                     win.set_similar_images_groups(ModelRc::new(VecModel::from(groups)));
 
-                    win.global::<AppState>().set_gallery_scroll_y(0.0);
-
                     let mut cancel_guard = thumb_cancel.lock().unwrap();
                     cancel_guard.store(true, Ordering::Relaxed);
                     let new_cancel = Arc::new(AtomicBool::new(false));
@@ -427,64 +425,6 @@ fn run_app_inner(
     wire_licenses_popup(&window);
     wire_save_settings_now(&window, included_dirs.clone(), excluded_dirs.clone(), referenced_dirs.clone());
 
-    let gallery_momentum: Rc<std::cell::RefCell<f32>> = Rc::new(std::cell::RefCell::new(0.0));
-
-    {
-        let weak_g = window.as_weak();
-        let mom = Rc::clone(&gallery_momentum);
-        window.global::<AppState>().on_gallery_swiped(move |total_delta, vel_px| {
-            if let Some(win) = weak_g.upgrade() {
-                let current = win.global::<AppState>().get_gallery_scroll_y();
-                let max_scroll = win.global::<AppState>().get_gallery_max_scroll_f();
-
-                let new_y = (current + total_delta).clamp(-max_scroll, 0.0);
-                win.global::<AppState>().set_gallery_scroll_y(new_y);
-
-                const VEL_THRESHOLD: f32 = 4.0;
-                *mom.borrow_mut() = if vel_px.abs() < VEL_THRESHOLD { 0.0 } else { vel_px * 1.5 };
-            }
-        });
-    }
-
-    {
-        let mom = Rc::clone(&gallery_momentum);
-        window.global::<AppState>().on_gallery_stop_momentum(move || {
-            *mom.borrow_mut() = 0.0;
-        });
-    }
-
-    let scroll_timer = Timer::default();
-    {
-        let weak_sc = window.as_weak();
-        let mom_sc = Rc::clone(&gallery_momentum);
-        scroll_timer.start(TimerMode::Repeated, std::time::Duration::from_millis(16), move || {
-            // ── Gallery fling ─────────────────────────────────────────────
-            let mut mom = mom_sc.borrow_mut();
-            if mom.abs() > 0.3 {
-                if let Some(win) = weak_sc.upgrade() {
-                    let current = win.global::<AppState>().get_gallery_scroll_y();
-                    let max_scroll = win.global::<AppState>().get_gallery_max_scroll_f();
-                    let new_y = (current + *mom).clamp(-max_scroll, 0.0);
-                    win.global::<AppState>().set_gallery_scroll_y(new_y);
-
-                    *mom *= 0.95;
-
-                    if new_y >= 0.0 || new_y <= -max_scroll {
-                        *mom = 0.0;
-                    }
-                }
-            } else if let Some(win) = weak_sc.upgrade() {
-                let max_s = win.global::<AppState>().get_gallery_max_scroll_f();
-                let cur = win.global::<AppState>().get_gallery_scroll_y();
-                if max_s > 0.0 && cur < -max_s {
-                    win.global::<AppState>().set_gallery_scroll_y(-max_s);
-                } else if cur > 0.0 {
-                    win.global::<AppState>().set_gallery_scroll_y(0.0);
-                }
-            }
-        });
-    }
-
     let weak = window.as_weak();
     let thumb_rx = Rc::new(std::cell::RefCell::new(thumb_rx));
     let scan_gen_poll = scan_gen;
diff --git a/cedinia/src/settings/gui_settings_values.rs b/cedinia/src/settings/gui_settings_values.rs
@@ -133,12 +133,7 @@ impl StringComboBoxItems {
             ("smallest", SearchMode::SmallestFiles, DisplaySpec::Translatable("option_search_mode_smallest")),
         ]);
 
-        let big_files_count = Self::convert(&[
-            ("10", "10", 10usize),
-            ("100", "100", 100),
-            ("1000", "1000", 1000),
-            ("10000", "10000", 10000) // Too slowefor now
-        ]);
+        let big_files_count = Self::convert(&[("10", "10", 10usize), ("100", "100", 100), ("1000", "1000", 1000), ("10000", "10000", 10000)]);
 
         let similarity_preset = Self::convert_i18n(&[
             ("very_high", SimilarityPreset::VeryHigh, DisplaySpec::Translatable("option_similarity_very_high")),
diff --git a/cedinia/ui/app_state.slint b/cedinia/ui/app_state.slint
@@ -193,12 +193,6 @@ export global AppState {
 
     in-out property <bool> similar_images_gallery_mode: true;
 
-    in-out property <length> gallery_scroll_y:     0px;
-    in-out property <float>  gallery_max_scroll_f: 0.0;
-    // total_delta: full swipe displacement (px)  vel: EMA velocity at release (px per touch-event)
-    callback gallery_swiped(float, float);
-    callback gallery_stop_momentum();
-
 
     in-out property <string> diag_thumbnails_size: "-";
     in-out property <string> diag_app_cache_size:  "-";
diff --git a/cedinia/ui/momentum_scroll.slint b/cedinia/ui/momentum_scroll.slint
diff --git a/cedinia/ui/similar_images_gallery.slint b/cedinia/ui/similar_images_gallery.slint
@@ -3,7 +3,7 @@ import { ScanState, SimilarGroupCard } from "common.slint";
 import { AppState, SimilarImagesSettings } from "app_state.slint";
 import { StatusChip, TouchButton } from "components.slint";
 import { Translations } from "translations.slint";
-import { MomentumScrollView } from "momentum_scroll.slint";
+import { ListView } from "std-widgets.slint";
 
 component GalleryImageCell {
     in property <image>  thumbnail;
@@ -271,11 +271,9 @@ export component SimilarImagesGallery {
     property <string> ctx_open_folder:  "";
     property <string> ctx_item_name:    "";
 
-    // Set to true while any per-row horizontal SGR is active so the vertical
-    // MomentumScrollView does not steal the gesture during reversals.
-    // Cleared only in the horizontal sgr.swiped handler (on finger-up).
     property <bool> _h_locked: false;
 
+
     VerticalLayout {
         spacing: 0px;
 
@@ -303,15 +301,8 @@ export component SimilarImagesGallery {
         // Inline the scroller directly (like ResultsList does) so that
         // vertical-stretch: 1.0 is applied to a real element in the layout,
         // not wrapped inside an intermediate component.
-        if groups.length > 0 : gallery_scroll := MomentumScrollView {
+        if groups.length > 0 : gallery_scroll := ListView {
             vertical-stretch: 1.0;
-            scroll_y <=> AppState.gallery_scroll_y;
-            lock_vertical: root._h_locked;
-            swiped(d, v, m) => {
-                AppState.gallery_max_scroll_f = m;
-                AppState.gallery_swiped(d, v);
-            }
-            stop_momentum => { AppState.gallery_stop_momentum(); }
 
             for group in root.groups : VerticalLayout {
                 padding-left: 8px;
@@ -345,7 +336,7 @@ export component SimilarImagesGallery {
                     height: 244px;
                     swiped => {
                         // Horizontal gesture finished — release the lock so the
-                        // outer MomentumScrollView can handle verticals again.
+                        // outer ListView can handle verticals again.
                         root._h_locked = false;
                         self.committed_x = self.committed_x + self.current-position.x - self.pressed-position.x;
                         if self.committed_x > 0px { self.committed_x = 0px; }
@@ -377,8 +368,7 @@ export component SimilarImagesGallery {
                                 checked:         img.checked;
                                 is_reference:    img.is_reference;
                                 // Cancel long-press when either the horizontal or the
-                                // vertical (MomentumScrollView) SGR claims the gesture.
-                                any_sgr_swiping: sgr.swiping || gallery_scroll.is_swiping;
+                                any_sgr_swiping: sgr.swiping;
                                 cell_long_pressed(f, d, n) => {
                                     root.ctx_open_file   = f;
                                     root.ctx_open_folder = d;
@@ -390,7 +380,7 @@ export component SimilarImagesGallery {
                     }
                 }
 
-                // When the horizontal swipe is confirmed, lock the vertical SGR.
+                // Track horizontal swipe state.
                 // Safe: depends only on sgr.swiping — no layout properties.
                 property <bool> _row_swiping: sgr.swiping;
                 changed _row_swiping => {
diff --git a/krokiet/AGENTS.md b/krokiet/AGENTS.md
