summaryrefslogtreecommitdiff
path: root/app/src/main/java/org/terst/feedreader/ui/ReaderScreen.kt
blob: 4becea696966c9e115b3b911d015b2b14422a482 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
package org.terst.feedreader.ui

import android.content.Intent
import android.net.Uri
import androidx.compose.foundation.ExperimentalFoundationApi
import androidx.compose.foundation.gestures.detectHorizontalDragGestures
import androidx.compose.foundation.layout.Box
import androidx.compose.foundation.layout.Column
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.foundation.layout.padding
import androidx.compose.foundation.pager.HorizontalPager
import androidx.compose.foundation.pager.rememberPagerState
import androidx.compose.material.icons.Icons
import androidx.compose.material.icons.filled.Article
import androidx.compose.material.icons.filled.ArrowBack
import androidx.compose.material.icons.filled.Bookmark
import androidx.compose.material.icons.filled.BookmarkBorder
import androidx.compose.material.icons.filled.CloudDone
import androidx.compose.material.icons.filled.CloudUpload
import androidx.compose.material.icons.filled.MoreVert
import androidx.compose.material.icons.filled.OpenInBrowser
import androidx.compose.material.icons.outlined.Article
import androidx.compose.material3.CircularProgressIndicator
import androidx.compose.material3.DropdownMenu
import androidx.compose.material3.DropdownMenuItem
import androidx.compose.material3.ExperimentalMaterial3Api
import androidx.compose.material3.Icon
import androidx.compose.material3.IconButton
import androidx.compose.material3.MaterialTheme
import androidx.compose.material3.Scaffold
import androidx.compose.material3.SnackbarHost
import androidx.compose.material3.SnackbarHostState
import androidx.compose.material3.Text
import androidx.compose.material3.TopAppBar
import androidx.compose.runtime.Composable
import androidx.compose.runtime.LaunchedEffect
import androidx.compose.runtime.getValue
import androidx.compose.runtime.mutableStateOf
import androidx.compose.runtime.remember
import androidx.compose.runtime.rememberCoroutineScope
import androidx.compose.runtime.setValue
import androidx.compose.ui.Alignment
import androidx.compose.ui.Modifier
import androidx.compose.ui.input.pointer.pointerInput
import androidx.compose.ui.platform.LocalContext
import androidx.compose.ui.platform.LocalDensity
import androidx.compose.ui.text.style.TextOverflow
import androidx.compose.ui.unit.dp
import androidx.compose.ui.viewinterop.AndroidView
import kotlinx.coroutines.flow.first
import kotlinx.coroutines.launch
import org.terst.feedreader.data.AppDatabase
import org.terst.feedreader.data.ArticleEntity
import org.terst.feedreader.data.FeedEntity
import org.terst.feedreader.data.Net
import org.terst.feedreader.data.Settings
import org.terst.feedreader.readability.ReaderExtractor
import org.terst.feedreader.stash.StashClient
import org.terst.feedreader.stash.StashSaveResult
import java.net.URLEncoder

// Below this length (of the plain feed-provided HTML), assume it's a teaser/summary,
// not the real article body, and auto-fall-back to on-device Readability extraction.
private const val THIN_CONTENT_THRESHOLD = 500

/**
 * Swipeable wrapper: snapshots the article-id order of the list this reader
 * was opened from (Unread/Saved/a specific feed) ONCE at entry, then pages
 * across that fixed queue. Deliberately a snapshot, not a live Flow -- opening
 * an article marks it read, which would otherwise drop it out of the live
 * Unread query mid-swipe and shift every subsequent page underneath the user.
 */
@OptIn(ExperimentalFoundationApi::class)
@Composable
fun ReaderScreen(
    feedId: String,
    initialArticleId: String,
    onBack: () -> Unit,
    onOpenLink: (String) -> Unit,
    onOpenBrowser: (String) -> Unit
) {
    val context = LocalContext.current
    val db = remember { AppDatabase.get(context) }
    var articleIds by remember(feedId) { mutableStateOf<List<String>?>(null) }

    LaunchedEffect(feedId) {
        val cutoff = System.currentTimeMillis() - RECENT_WINDOW_MS
        val flow = when (feedId) {
            UNREAD_FEED_ID -> db.articleDao().observeUnread(cutoff)
            OLDER_FEED_ID -> db.articleDao().observeOlderUnread(cutoff)
            SAVED_FEED_ID -> db.articleDao().observeSaved()
            else -> db.articleDao().observeByFeed(feedId)
        }
        articleIds = flow.first().map { it.id }
    }

    val ids = articleIds
    if (ids == null) {
        Box(modifier = Modifier.fillMaxSize()) {
            CircularProgressIndicator(modifier = Modifier.align(Alignment.Center))
        }
        return
    }
    // The article may have been opened from somewhere not in this queue (shouldn't normally
    // happen, but fall back to a single-item queue rather than crashing on indexOfFirst = -1).
    val safeIds = if (ids.contains(initialArticleId)) ids else listOf(initialArticleId)
    val initialIndex = safeIds.indexOf(initialArticleId).coerceAtLeast(0)
    val pagerState = rememberPagerState(initialPage = initialIndex) { safeIds.size }
    val scope = rememberCoroutineScope()
    val swipeThresholdPx = with(LocalDensity.current) { 80.dp.toPx() }

    // HorizontalPager's own full-surface drag detection was grabbing 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.
    // detectHorizontalDragGestures has its own orientation-aware touch-slop detection, so
    // it already declines to engage for vertical-dominant movement -- confirmed live: a
    // diagonal mid-screen scroll stays vertical, doesn't page-turn. (A generic,
    // non-oriented detectDragGestures was tried for a stricter dx/dy ratio check and broke
    // scrolling entirely: its slop detector claims the first bit of movement in ANY
    // direction isotropically, before it knows which way the gesture is really going --
    // reported live as "the article view doesn't scroll at all anymore.") Also: page-turn
    // commit is based on total drag distance, not where the gesture started, since an
    // edge-anchored version collided with Android's own edge-swipe-back gesture (confirmed
    // live: an edge swipe triggered system back, not the pager).
    Box(
        modifier = Modifier
            .fillMaxSize()
            .pointerInput(safeIds.size) {
                var totalDrag = 0f
                detectHorizontalDragGestures(
                    onDragStart = { totalDrag = 0f },
                    onDragCancel = { totalDrag = 0f },
                    onDragEnd = {
                        val direction = when {
                            totalDrag <= -swipeThresholdPx -> 1
                            totalDrag >= swipeThresholdPx -> -1
                            else -> 0
                        }
                        if (direction != 0) {
                            val target = (pagerState.currentPage + direction).coerceIn(0, safeIds.size - 1)
                            scope.launch { pagerState.animateScrollToPage(target) }
                        }
                    },
                    onHorizontalDrag = { change, dragAmount ->
                        change.consume()
                        totalDrag += dragAmount
                    }
                )
            }
    ) {
        HorizontalPager(state = pagerState, userScrollEnabled = false, modifier = Modifier.fillMaxSize()) { page ->
            ReaderContent(
                articleId = safeIds[page],
                onBack = onBack,
                onOpenLink = onOpenLink,
                onOpenBrowser = onOpenBrowser
            )
        }
    }
}

@OptIn(ExperimentalMaterial3Api::class)
@Composable
private fun ReaderContent(
    articleId: String,
    onBack: () -> Unit,
    onOpenLink: (String) -> Unit,
    onOpenBrowser: (String) -> Unit
) {
    val context = LocalContext.current
    val db = remember { AppDatabase.get(context) }
    val scope = rememberCoroutineScope()
    val snackbarHostState = remember { SnackbarHostState() }

    var article by remember(articleId) { mutableStateOf<ArticleEntity?>(null) }
    var feed by remember(articleId) { mutableStateOf<FeedEntity?>(null) }
    var feedContent by remember(articleId) { mutableStateOf<String?>(null) }
    var readerContent by remember(articleId) { mutableStateOf<String?>(null) }
    var isReaderMode by remember(articleId) { mutableStateOf(false) }
    var isExtracting by remember(articleId) { mutableStateOf(false) }
    var isSaving by remember(articleId) { mutableStateOf(false) }
    var moreMenuExpanded by remember(articleId) { mutableStateOf(false) }

    LaunchedEffect(articleId) {
        val loaded = db.articleDao().getById(articleId) ?: return@LaunchedEffect
        article = loaded
        db.articleDao().setRead(articleId, true)
        val loadedFeed = loaded.feedId?.let { db.feedDao().getById(it) }
        feed = loadedFeed

        feedContent = loaded.feedContentHtml
        readerContent = loaded.extractedContent
        val plainLength = feedContent?.replace(Regex("<[^>]*>"), "")?.trim()?.length ?: 0

        if (loadedFeed?.readerModeDisabled != true && readerContent == null && plainLength < THIN_CONTENT_THRESHOLD) {
            // Feed content is too thin to be the real article -- auto-extract, since in this
            // case reader mode isn't really an alternative view, it's the only real content.
            isExtracting = true
            val extracted = ReaderExtractor.extract(context, loaded.link)
            isExtracting = false
            extracted?.content?.let {
                readerContent = it
                db.articleDao().setExtractedContent(articleId, it)
            }
            isReaderMode = true
        }
    }

    fun toggleReaderMode() {
        val a = article ?: return
        if (feed?.readerModeDisabled == true) return
        if (isReaderMode) {
            isReaderMode = false
            return
        }
        if (readerContent != null) {
            isReaderMode = true
            return
        }
        scope.launch {
            isExtracting = true
            val extracted = ReaderExtractor.extract(context, a.link)
            isExtracting = false
            val content = extracted?.content
            if (content != null) {
                readerContent = content
                db.articleDao().setExtractedContent(articleId, content)
                isReaderMode = true
            } else {
                snackbarHostState.showSnackbar("Couldn't extract a reader view for this article")
            }
        }
    }

    Scaffold(
        topBar = {
            TopAppBar(
                title = {
                    Column {
                        Text(article?.title.orEmpty(), maxLines = 1, overflow = TextOverflow.Ellipsis)
                        feed?.title?.let {
                            Text(
                                it,
                                maxLines = 1,
                                overflow = TextOverflow.Ellipsis,
                                style = MaterialTheme.typography.labelSmall,
                                color = MaterialTheme.colorScheme.onSurfaceVariant
                            )
                        }
                    }
                },
                navigationIcon = {
                    IconButton(onClick = onBack) {
                        Icon(Icons.Default.ArrowBack, contentDescription = "Back")
                    }
                },
                actions = {
                    val currentArticle = article
                    // Only offer the toggle when there's an actual feed-provided alternative to
                    // switch back to (if we already had to auto-extract for thin feed content,
                    // reader mode IS the only content) and the feed hasn't opted out of it.
                    if (feedContent != null && feed?.readerModeDisabled != true) {
                        IconButton(enabled = !isExtracting, onClick = { toggleReaderMode() }) {
                            Icon(
                                if (isReaderMode) Icons.Filled.Article else Icons.Outlined.Article,
                                contentDescription = "Toggle reader view"
                            )
                        }
                    }
                    IconButton(onClick = {
                        val a = currentArticle ?: return@IconButton
                        val browserIntent = Intent(Intent.ACTION_VIEW, Uri.parse(a.link))
                        context.startActivity(browserIntent)
                    }) {
                        Icon(Icons.Default.OpenInBrowser, contentDescription = "Open in browser")
                    }
                    // Purely local: just flips isSaved (shows up in the Saved list). Does NOT
                    // touch Stash -- that's a separate, explicit action below, since archiving
                    // to Stash is a heavier/networked action the user should trigger on purpose.
                    IconButton(onClick = {
                        val a = currentArticle ?: return@IconButton
                        val newSaved = !a.isSaved
                        article = a.copy(isSaved = newSaved)
                        scope.launch { db.articleDao().setSaved(a.id, newSaved) }
                    }) {
                        Icon(
                            if (currentArticle?.isSaved == true) Icons.Default.Bookmark else Icons.Default.BookmarkBorder,
                            contentDescription = "Save locally"
                        )
                    }
                    IconButton(
                        enabled = !isSaving,
                        onClick = {
                            val a = currentArticle ?: return@IconButton
                            isSaving = true
                            scope.launch {
                                val settings = Settings(context)
                                val client = StashClient(Net.client, settings.stashBaseUrlNow(), settings.stashApiKeyNow())
                                when (val result = client.saveArticle(a.link, a.title)) {
                                    is StashSaveResult.Saved -> {
                                        db.articleDao().setStashArtifactId(a.id, result.artifact.id)
                                        article = a.copy(stashArtifactId = result.artifact.id)
                                        snackbarHostState.showSnackbar("Saved to Stash")
                                    }
                                    is StashSaveResult.AlreadySaved -> {
                                        db.articleDao().setStashArtifactId(a.id, result.existingId)
                                        article = a.copy(stashArtifactId = result.existingId)
                                        snackbarHostState.showSnackbar("Already in Stash")
                                    }
                                    is StashSaveResult.Failed ->
                                        snackbarHostState.showSnackbar("Save failed: ${result.message}")
                                }
                                isSaving = false
                            }
                        }
                    ) {
                        Icon(
                            if (currentArticle?.stashArtifactId != null) Icons.Default.CloudDone else Icons.Default.CloudUpload,
                            contentDescription = "Save to Stash"
                        )
                    }
                    IconButton(onClick = { moreMenuExpanded = true }) {
                        Icon(Icons.Default.MoreVert, contentDescription = "More")
                    }
                    DropdownMenu(expanded = moreMenuExpanded, onDismissRequest = { moreMenuExpanded = false }) {
                        // A real (JS-on) browser, not the extraction pipeline -- search results
                        // and login pages need actual interaction, not a Readability pass.
                        DropdownMenuItem(
                            text = { Text("Find another source") },
                            onClick = {
                                moreMenuExpanded = false
                                val a = currentArticle ?: return@DropdownMenuItem
                                val query = URLEncoder.encode(a.title, "UTF-8")
                                onOpenBrowser("https://www.google.com/search?q=$query")
                            }
                        )
                        DropdownMenuItem(
                            text = { Text("Log in to this site") },
                            onClick = {
                                moreMenuExpanded = false
                                val a = currentArticle ?: return@DropdownMenuItem
                                onOpenBrowser(a.link)
                            }
                        )
                    }
                }
            )
        },
        snackbarHost = { SnackbarHost(snackbarHostState) }
    ) { padding ->
        Box(modifier = Modifier.fillMaxSize().padding(padding)) {
            val content = if (isReaderMode) readerContent else (feedContent ?: readerContent)
            val baseUrl = article?.link
            if (content != null) {
                // AndroidView's update lambda reruns on every recomposition of this
                // composable -- including ones unrelated to content, like isSaving
                // flipping while a Stash save is in flight. Reloading the WebView's HTML
                // on those unrelated recompositions was resetting scroll position mid-swipe,
                // which is what made scrolling feel unsmooth/janky. Track what's actually
                // loaded and only reload when the content string itself changes.
                val loadedContent = remember(articleId) { mutableStateOf<String?>(null) }
                AndroidView(
                    modifier = Modifier.fillMaxSize(),
                    factory = { ctx ->
                        newReaderWebView(ctx, onOpenLink).apply {
                            loadDataWithBaseURL(baseUrl, wrapReaderHtml(content), "text/html", "UTF-8", null)
                            loadedContent.value = content
                        }
                    },
                    update = { webView ->
                        if (loadedContent.value != content) {
                            webView.loadDataWithBaseURL(baseUrl, wrapReaderHtml(content), "text/html", "UTF-8", null)
                            loadedContent.value = content
                        }
                    },
                    // AndroidView does not destroy() the underlying WebView on its own --
                    // without this, swiping through many articles (each page holding its own
                    // WebView) leaks a WebView's native resources per page and eventually
                    // exhausts memory over a long reading session.
                    onRelease = { webView -> webView.destroy() }
                )
            } else if (isExtracting) {
                CircularProgressIndicator(modifier = Modifier.align(Alignment.Center))
            }
        }
    }
}