fix(mobile): no marking read when no scrolling (#2913)

RSSNext · Feb 28, 2025 · cd576ff · cd576ff
1 parent 34540a1
commit cd576ff
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/apps/mobile/src/modules/entry-list/hooks.ts b/apps/mobile/src/modules/entry-list/hooks.ts
@@ -12,7 +12,7 @@ export function useOnViewableItemsChanged({
 }: {
   idExtractor?: (item: ViewToken) => string
 } = {}) {
-  const orientation = useRef<"down" | "up">("down")
+  const orientation = useRef<"down" | "up" | "initial">("initial")
   const lastOffset = useRef(0)
 
   const markAsReadWhenScrolling = useGeneralSettingKey("scrollMarkUnread")
@@ -26,15 +26,17 @@ export function useOnViewableItemsChanged({
   }) => void = useNonReactiveCallback(({ viewableItems, changed }) => {
     debouncedFetchEntryContentByStream(viewableItems.map((item) => stableIdExtractor(item)))
 
-    if (markAsReadWhenScrolling && orientation.current === "down") {
+    if (orientation.current !== "down") return
+
+    if (markAsReadWhenScrolling) {
       changed
         .filter((item) => !item.isViewable)
         .forEach((item) => {
           unreadSyncService.markEntryAsRead(stableIdExtractor(item))
         })
     }
 
-    if (markAsReadWhenRendering && orientation.current === "down") {
+    if (markAsReadWhenRendering) {
       viewableItems.forEach((item) => {
         unreadSyncService.markEntryAsRead(stableIdExtractor(item))
       })