From 465c24ed3a244b4a6f966c0cccedcca224ae058d Mon Sep 17 00:00:00 2001
From: Gigi <dergigi@pm.me>
Date: Sat, 25 Oct 2025 00:52:49 +0200
Subject: [PATCH] fix: resolve highlight loading issues for articles

- Add missing eventStore parameter to fetchHighlightsForArticle call
- Clear highlights immediately when starting to load new article
- Fix infinite loading spinners when articles have zero highlights
- Ensure highlights are properly stored and persisted
---
 src/hooks/useArticleLoader.ts | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/hooks/useArticleLoader.ts b/src/hooks/useArticleLoader.ts
index 68b9ffa3..08e3c83e 100644
--- a/src/hooks/useArticleLoader.ts
+++ b/src/hooks/useArticleLoader.ts
@@ -76,6 +76,10 @@ export function useArticleLoader({
       setSelectedUrl(`nostr:${naddr}`)
       setIsCollapsed(true)
       
+      // Clear highlights immediately when starting to load a new article
+      setHighlights([])
+      setHighlightsLoading(false) // Don't show loading yet
+      
       // If we have preview data from navigation, show it immediately (no skeleton!)
       if (previewData) {
         setReaderContent({
@@ -251,7 +255,9 @@ export function useArticleLoader({
                   return next.sort((a, b) => b.created_at - a.created_at)
                 })
               },
-              settingsRef.current
+              settingsRef.current,
+              false, // force
+              eventStore || undefined
             )
           } else {
             // No article event to fetch highlights for - clear and don't show loading