From a7937cd657b7717fe7cd218da96afbe8b99c9583 Mon Sep 17 00:00:00 2001
From: Huang Xin <chrox.huang@gmail.com>
Date: Mon, 24 Nov 2025 14:47:03 +0800
Subject: [PATCH] fix(translator): resolve occasional translation failures when
 navigating to new chapters (#2527)

Closes #2451.
---
 .../app/reader/hooks/useTextTranslation.ts    | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/apps/readest-app/src/app/reader/hooks/useTextTranslation.ts b/apps/readest-app/src/app/reader/hooks/useTextTranslation.ts
index 02a265b4..8d511924 100644
--- a/apps/readest-app/src/app/reader/hooks/useTextTranslation.ts
+++ b/apps/readest-app/src/app/reader/hooks/useTextTranslation.ts
@@ -13,9 +13,8 @@ export function useTextTranslation(
   widthLineBreak = false,
   targetBlockClassName = 'translation-target-block',
 ) {
-  const { getViewSettings, getViewState, getProgress } = useReaderStore();
+  const { getViewSettings, getProgress } = useReaderStore();
   const viewSettings = getViewSettings(bookKey);
-  const viewState = getViewState(bookKey);
   const progress = getProgress(bookKey);
 
   const enabled = useRef(viewSettings?.translationEnabled);
@@ -55,7 +54,11 @@ export function useTextTranslation(
     const observer = createTranslationObserver();
     observerRef.current = observer;
     const nodes = walkTextNodes(view, ['pre', 'code', 'math']);
-    console.log('Observing text nodes for translation:', nodes.length);
+    console.log(
+      'Observing text nodes for translation:',
+      nodes.length,
+      // nodes.map((n) => n.textContent),
+    );
     allTextNodes.current = nodes;
     nodes.forEach((el) => observer.observe(el));
   };
@@ -250,8 +253,10 @@ export function useTextTranslation(
         console.log('Range not found in text nodes');
         return;
       }
-      const beforeStart = Math.max(0, startIndex - 2);
-      const afterEnd = Math.min(nodes.length - 1, endIndex + 2);
+      const beforeContext = 2;
+      const afterContext = 5;
+      const beforeStart = Math.max(0, startIndex - beforeContext);
+      const afterEnd = Math.min(nodes.length - 1, endIndex + afterContext);
       for (let i = beforeStart; i <= afterEnd; i++) {
         const node = nodes[i];
         if (node) {
@@ -263,12 +268,12 @@ export function useTextTranslation(
   );
 
   useEffect(() => {
-    if (enabled.current && viewState?.ttsEnabled && progress && document.hidden) {
+    if (enabled.current && progress) {
       const { range } = progress;
       translateInRange(range);
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [viewState?.ttsEnabled, progress]);
+  }, [progress]);
 
   useEffect(() => {
     if (!viewSettings) return;