fix: auto view source content fail, close #4119

Author: hyoban

apps/desktop/layer/renderer/src/hooks/biz/useNavigateEntry.ts

@@ -2,9 +2,9 @@ import { getReadonlyRoute, getStableRouterNavigate } from "@follow/components/at
 import { useMobile } from "@follow/components/hooks/useMobile.js"
 import { useSheetContext } from "@follow/components/ui/sheet/context.js"
 import type { FeedViewType } from "@follow/constants"
+import { getEntry } from "@follow/store/entry/getter"
 import { getSubscriptionByFeedId } from "@follow/store/subscription/getter"
 import { tracker } from "@follow/tracker"
-import { nextFrame } from "@follow/utils"
 import { useCallback } from "react"
 import { toast } from "sonner"
 
@@ -129,10 +129,10 @@ export const navigateEntry = (options: NavigateEntryOptions) => {
 
   disableShowAISummaryOnce()
   disableShowAITranslationOnce()
-
-  nextFrame(() => {
+  const sourceContent = getEntry(parsedOptions.entryId)?.settings?.sourceContent
+  if (!sourceContent) {
     resetShowSourceContent()
-  })
+  }
 
   const navigate = getStableRouterNavigate()
 

apps/desktop/layer/renderer/src/modules/entry-content/ApplyEntryActions.tsx

@@ -1,5 +1,5 @@
 import { useEntry } from "@follow/store/entry/hooks"
-import { useEffect, useRef } from "react"
+import { useEffect } from "react"
 
 import { enableShowSourceContent } from "~/atoms/source-content"
 import { enableEntryReadability } from "~/hooks/biz/useEntryActions"
@@ -18,22 +18,19 @@ export const ApplyEntryActions = ({ entryId }: { entryId: string }) => {
     }
   })
 
-  useViewSourceContentAutoToggleEffect(entry?.sourceContent)
-
   return (
-    <>{entry?.readability && <ReadabilityAutoToggleEffect id={entryId} url={entry?.url ?? ""} />}</>
+    <>
+      {entry?.sourceContent && <ViewSourceContentAutoToggleEffect id={entryId} />}
+      {entry?.readability && <ReadabilityAutoToggleEffect id={entryId} url={entry?.url ?? ""} />}
+    </>
   )
 }
 
-const useViewSourceContentAutoToggleEffect = (when: boolean | undefined) => {
-  const onceRef = useRef(false)
+const ViewSourceContentAutoToggleEffect = ({ id }: { id: string }) => {
   useEffect(() => {
-    if (!when) return
-    if (!onceRef.current) {
-      onceRef.current = true
-      enableShowSourceContent()
-    }
-  }, [when])
+    enableShowSourceContent()
+  }, [id])
+  return null
 }
 
 export const ReadabilityAutoToggleEffect = ({ url, id }: { url: string; id: string }) => {