From bedf8a47a8b0601f4e3c253e4d2a7325c6ccbfc6 Mon Sep 17 00:00:00 2001
From: Peter Stone <thepeterstone@gmail.com>
Date: Wed, 22 Jul 2026 18:56:47 +0000
Subject: Fix accidental page-swipe while scrolling; bring linked-article
 reader to parity

- ReaderScreen: HorizontalPager's full-surface drag detection was treating
  ordinary vertical scrolling inside the embedded WebView (a native View,
  which doesn't arbitrate with Compose's gesture system the way a
  pure-Compose scrollable child would) as an attempted horizontal page-turn
  on anything less than a perfectly vertical swipe -- i.e. most real thumb
  scrolling. Tried gating it to edge-zone-only drags first, but that
  collides with Android's own edge-swipe-back gesture on gesture-nav
  devices (reproduced live: an edge swipe triggered system back, not the
  pager). Landed on disabling the Pager's built-in gesture and driving it
  from a custom detector that works anywhere on screen but only commits to
  a page turn once a drag is clearly horizontal-dominant (>2x the vertical
  movement) -- ordinary scrolling, even sloppy/diagonal, is left completely
  unconsumed for the WebView. Verified live: a deliberately diagonal
  mid-screen scroll now stays on the same article; a decisive horizontal
  swipe still changes pages.

- LinkReaderScreen: was missing the bookmark/Stash/reader-toggle/menu
  actions entirely -- reported after tapping a link left "all my icons...
  gone." Brought to parity with the main article reader: save locally and
  save to Stash now lazily create a standalone ArticleEntity (feedId=null,
  id = hash of the URL, same scheme ShareReceiverActivity uses) on first
  use; reader-mode toggle falls back to the live page (JS on) instead of
  always forcing extraction; same login/find-another-source menu. Verified
  live: all 5 icons present once extraction finishes, bookmarking a linked
  article correctly creates a standalone DB row.
---
 app/src/main/java/org/terst/feedreader/ui/MainActivity.kt | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'app/src/main/java/org/terst/feedreader/ui/MainActivity.kt')

diff --git a/app/src/main/java/org/terst/feedreader/ui/MainActivity.kt b/app/src/main/java/org/terst/feedreader/ui/MainActivity.kt
index bd09393..43e1680 100644
--- a/app/src/main/java/org/terst/feedreader/ui/MainActivity.kt
+++ b/app/src/main/java/org/terst/feedreader/ui/MainActivity.kt
@@ -103,7 +103,8 @@ fun FeedReaderApp() {
             LinkReaderScreen(
                 url = url,
                 onBack = { navController.popBackStack() },
-                onOpenLink = { nextUrl -> navController.navigate("linkReader/${encodeNavArg(nextUrl)}") }
+                onOpenLink = { nextUrl -> navController.navigate("linkReader/${encodeNavArg(nextUrl)}") },
+                onOpenBrowser = { browserUrl -> navController.navigate("browser/${encodeNavArg(browserUrl)}") }
             )
         }
         composable(
-- 
cgit v1.2.3