Fixing #9

Quite a lot of changes in order to achieve this:
- had to activate the navigation link programmatically in order to capture single and double tap
- NavigationLink activated based on new view state for selected post tag
- moved the RequestView to the ContentView so the selected post wouldn't reload the whole PostList and retriever the request
- PostList now initialised with Listing object
- setting the contentShape of the PostView in the list in order to capture hits in the whole area, otherwise it would land on the NavigationLink behind
- adding two tap gestures in order of precedence
- set the default frame for the new spinner to guarantee a minimum size while loading

The only secondary effect from this solution I wasn't able to fix is that now cells don't get a selected state like before. I accept suggestions.

Author: Fábio Oliveira

Reddit-macOS/ContentView.swift

@@ -22,8 +22,16 @@ struct ContentView : View {
     var body: some View {
         NavigationView {
             /// Load the posts
-            PostList(subreddit: state.subreddit, sortBy: state.sortBy)
-                .frame(minWidth: 300)
+            RequestView(Listing.self, Request {
+                Url(API.subredditURL(state.subreddit, sortBy))
+                Query(["raw_json":"1"])
+            }) { listing in
+                PostList(listing: listing, subreddit: self.state.subreddit, sortBy: self.state.sortBy)
+                    .frame(minWidth: 300)
+                /// Spinner when loading
+                SpinnerView()
+                    .frame(minWidth: 300, minHeight: 300)
+            }
             Text("Select a post")
                 .frame(maxWidth: .infinity, maxHeight: .infinity)
         }

Reddit-macOS/Views/PostList.swift

@@ -10,33 +10,36 @@ import SwiftUI
 import Request
 
 struct PostList: View {
+    let listing: Listing?
     let subreddit: String
     let sortBy: SortBy
 
+    @State private var selection: String? = nil
+    
     var body: some View {
         List {
             Section(header: Text("\(subreddit) | \(sortBy.rawValue)")) {
-                /// Load posts from web and decode as `Listing`
-                RequestView(Listing.self, Request {
-                    Url(API.subredditURL(subreddit, sortBy))
-                    Query(["raw_json":"1"])
-                }) { listing in
-                    /// List of `PostView`s when loaded
-                    ForEach(listing != nil ? listing!.data.children.map { $0.data } : []) { post in
-                        NavigationLink(destination: PostDetailView(post: post)) {
-                            PostView(post: post)
+                /// List of `PostView`s when loaded
+                ForEach(listing != nil ? listing!.data.children.map { $0.data } : []) { post in
+                    VStack {
+                        NavigationLink(destination: PostDetailView(post: post), tag: post.id, selection: self.$selection) { EmptyView() }
+                        PostView(post: post)
                             .tag(post.id)
                             .padding(EdgeInsets(top: 5, leading: 0, bottom: 5, trailing: 0))
+                            .contentShape(Rectangle())
                             /// Double-click to open a new window for the `PostDetailView`
                             .onTapGesture(count: 2) {
-                                let controller = DetailWindowController(rootView: PostDetailView(post: post))
+                                let detailView = PostDetailView(post: post)
+                                
+                                let controller = DetailWindowController(rootView: detailView)
                                 controller.window?.title = post.title
                                 controller.showWindow(nil)
-                            }
+                        }
+                            /// Adding after the double tap so that double tap takes precedence
+                        .onTapGesture(count: 1) {
+                            self.selection = post.id
                         }
                     }
-                    /// Spinner when loading
-                    SpinnerView()
                 }
             }
         }