From c99f1c7b77efa20c37b288c028eb74b9622b45a4 Mon Sep 17 00:00:00 2001
From: archipelago <archipelago@localhost>
Date: Wed, 29 Jul 2026 04:32:41 -0400
Subject: [PATCH] fix(ui): transactions modal touch scrolling on phones
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The tx list kept a vestigial overflow-y-auto from before the modal
contract refactor made BaseModal's slot wrapper the scroller. With a
modal open, modal-scroll-locked applies overscroll-behavior:contain to
every .overflow-y-auto inside the overlay, so touch scrolls latched
onto the non-scrollable inner list and could not chain up to the real
scroller — the modal was unscrollable on any touch device. Wheel input
latches onto the scrollable ancestor directly, which is why desktop
never showed it. Verified with headless-chromium touch synthesis at
360x640/320x568: list scrolls to bottom, background stays contained.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 neode-ui/src/components/TransactionsModal.vue | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/neode-ui/src/components/TransactionsModal.vue b/neode-ui/src/components/TransactionsModal.vue
index 9f8424db..efa2b84c 100644
--- a/neode-ui/src/components/TransactionsModal.vue
+++ b/neode-ui/src/components/TransactionsModal.vue
@@ -26,7 +26,12 @@
           <p class="text-white/40 text-sm">No {{ activeFilter }} transactions</p>
         </div>
 
-        <div v-else class="flex-1 overflow-y-auto -mx-2 px-2 divide-y divide-white/5">
+        <!-- Must NOT be its own scroll container: BaseModal's slot wrapper is the
+             scroller, and modal-scroll-locked gives every .overflow-y-auto inside
+             the overlay overscroll-behavior:contain — a nested scroller with no
+             overflow then swallows touch scrolling entirely (phones only; wheel
+             latches onto the scrollable ancestor and never sees the bug). -->
+        <div v-else class="-mx-2 px-2 divide-y divide-white/5">
           <div
             v-for="tx in filteredTransactions"
             :key="(tx.kind || 'onchain') + tx.tx_hash + tx.time_stamp"