Merge pull request #324 from weiran/claude/feed-category-loading-state-011CUhjzTbEhavoXDwAGfgv1

Show loading state when switching category

Author: weiran

Features/Feed/Sources/Feed/FeedView.swift

@@ -92,6 +92,10 @@ public struct FeedView<NavigationStore: NavigationStoreProtocol>: View {
                     searchText = newValue
                 }
             }
+            .onChange(of: selectedPostType) { _ in
+                // Clear selection when category changes to prevent stale sidebar selection
+                selectedPostId = nil
+            }
         .task { @Sendable in
             // Set the navigation store for the voting view model
             votingViewModel.navigationStore = navigationStore
@@ -121,7 +125,7 @@ public struct FeedView<NavigationStore: NavigationStoreProtocol>: View {
         Group {
             if viewModel.hasActiveSearch {
                 searchContentView
-            } else if viewModel.isLoading, viewModel.posts.isEmpty {
+            } else if viewModel.isLoading && viewModel.posts.isEmpty && viewModel.postType != .bookmarks {
                 AppLoadingStateView(message: "Loading...")
             } else if shouldShowBookmarksEmptyState {
                 AppEmptyStateView(
@@ -172,6 +176,7 @@ public struct FeedView<NavigationStore: NavigationStoreProtocol>: View {
         .if(isSidebar) { view in view.listStyle(.sidebar) }
         .if(!isSidebar) { view in view.listStyle(.plain) }
         .frame(maxWidth: .infinity, maxHeight: .infinity)
+        .id(selectedPostType)
     }
 
     @ViewBuilder

Features/Feed/Sources/Feed/FeedViewModel.swift

@@ -213,11 +213,15 @@ public final class FeedViewModel: @unchecked Sendable {
 
         postType = newType
         persistLastFeedCategoryIfNeeded()
-        await refreshFeed()
+        reset(clearPosts: true)  // Clear posts immediately to prevent flash of old data
+        await feedLoader.refresh()
     }
 
     @MainActor
-    private func reset() {
+    private func reset(clearPosts: Bool = false) {
+        if clearPosts {
+            feedLoader.data = []
+        }
         postIds = Set()
         pageIndex = 1
         lastPostId = 0