minor changes

Author: nickxiao-zoom

.gitignore

@@ -0,0 +1,3 @@
+V2exOS.xcodeproj/project.xcworkspace/xcuserdata/
+V2exOS.xcodeproj/xcuserdata/*.xcuserdatad/
+.python-version

V2exOS/Views/TopicDetail/TopicDetailView.swift

@@ -17,11 +17,11 @@ struct TopicDetailView: View {
 
   @State var commentList: [V2Comment]?
   @State var page = 0
-  @State var commenEnd = false
+  @State var commentEnd = false
   @State var isCommentLoading = false
 
-  func hasCommen() -> Bool {
-    return currentUser.user != nil && (commenEnd || commentList?.count ?? 0 < topic.replies ?? 0)
+  func hasMoreComments() -> Bool {
+    return currentUser.user != nil && (commentEnd || commentList?.count ?? 0 < topic.replies ?? 0)
   }
 
   var body: some View {
@@ -64,12 +64,16 @@ struct TopicDetailView: View {
 
       CommentListView(commentCount: topic.replies, commentList: commentList)
 
-      if hasCommen() {
+      if hasMoreComments() {
         Spacer()
-        ProgressView()
-          .onAppear {
-            loadComments(page: page + 1)
-          }
+        HStack {
+          Spacer()
+          ProgressView()
+            .onAppear {
+              loadComments(page: page + 1)
+            }
+          Spacer()
+        }
       }
     }
     .foregroundColor(Color(NSColor.labelColor))
@@ -92,20 +96,20 @@ struct TopicDetailView: View {
           commentList = res?.result
         }else{
           if let list = res?.result {
-            if !hasCommen() {
+            if !hasMoreComments() {
               isCommentLoading = false
               return
             }
 
             commentList?.append(contentsOf: list)
             // 到底了
             if list.count == 0 {
-              commenEnd = true
+              commentEnd = true
             }
           }
         }
       } catch {
-        commenEnd = true
+        commentEnd = true
         print(error)
       }
 

V2exOS/Views/TopicList/TopicListCellView.swift

@@ -10,38 +10,49 @@ import V2exAPI
 import Kingfisher
 
 struct TopicListCellView: View {
+  @State var isMemberLoading = false
 
   @Environment(\.colorScheme) var colorScheme
 
   let topic : V2Topic
+  @State var member : V2Member?
+  
+  init(topic: V2Topic) {
+    self.topic = topic
+    self.isMemberLoading = topic.member == nil
+  }
 
   var body: some View {
     NavigationLink {
       TopicDetailView(topic: topic)
     } label: {
 
       HStack {
-        if let avatarUrl = topic.member?.avatarLarge {
-          KFImage.url(URL(string: avatarUrl))
-            .resizable()
-            .fade(duration: 0.25)
-            .scaledToFit()
-            .frame(width: 48, height: 48)
-            .mask(RoundedRectangle(cornerRadius: 8))
+        
+        if !isMemberLoading {
+          let user = topic.member ?? member
+          if let avatarUrl = user?.avatarLarge {
+            KFImage.url(URL(string: avatarUrl))
+              .resizable()
+              .fade(duration: 0.25)
+              .scaledToFit()
+              .frame(width: 48, height: 48)
+              .mask(RoundedRectangle(cornerRadius: 8))
+          }
         }
+        
         VStack(alignment: .leading, spacing: 6) {
 
           Text(topic.title ?? "")
             .lineLimit(2)
 
           HStack() {
-            if let username = topic.member?.username {
+            
+            if let username = topic.member?.username ?? topic.lastReplyBy {
               UserName(username)
-              
               Text("•")
             }
 
-            
             if let lastModified = topic.lastModified {
               Text(Date.init(timeIntervalSince1970: TimeInterval(lastModified)).fromNow())
             }
@@ -60,6 +71,12 @@ struct TopicListCellView: View {
 
       }
       .foregroundColor(Color(NSColor.labelColor))
+      .task {
+        if let name = topic.lastReplyBy {
+          member = try? await v2ex.memberShow(username: name)
+          isMemberLoading = member == nil
+        }
+      }
     }
   }
 }

V2exOS/Views/TopicList/TopicListView.swift

@@ -12,7 +12,6 @@ import Kingfisher
 struct TopicListView: View {
 
   var nodeName: String
-//  var node: V2Node?
 
   @State var isLoading = true
   @State var topics: [V2Topic]?
@@ -33,12 +32,16 @@ struct TopicListView: View {
             }
 
             if topics.count > 0 && nodeName != "ALL" && nodeName != "HOT" {
-              ProgressView()
-                .onAppear {
-                  Task {
-                    await self.loadData(page: self.page + 1)
+              HStack {
+                Spacer()
+                ProgressView()
+                  .onAppear {
+                    Task {
+                      await self.loadData(page: self.page + 1)
+                    }
                   }
-                }
+                Spacer()
+              }
             }
           }