Update .gitignore, refactor Swift views for improved UI and data binding

Author: androiddevnotes

.gitignore

@@ -1 +1,63 @@
-Secrets.plist
+Secrets.plist
+# Xcode
+#
+# gitignore contributors: remember to update Global/Xcode.gitignore, Objective-C.gitignore & Swift.gitignore
+
+## User settings
+xcuserdata/
+
+## Obj-C/Swift specific
+*.hmap
+
+## App packaging
+*.ipa
+*.dSYM.zip
+*.dSYM
+
+## Playgrounds
+timeline.xctimeline
+playground.xcworkspace
+
+# Swift Package Manager
+#
+# Add this line if you want to avoid checking in source code from Swift Package Manager dependencies.
+# Packages/
+# Package.pins
+# Package.resolved
+# *.xcodeproj
+#
+# Xcode automatically generates this directory with a .xcworkspacedata file and xcuserdata
+# hence it is not needed unless you have added a package configuration file to your project
+# .swiftpm
+
+.build/
+
+# CocoaPods
+#
+# We recommend against adding the Pods directory to your .gitignore. However
+# you should judge for yourself, the pros and cons are mentioned at:
+# https://guides.cocoapods.org/using/using-cocoapods.html#should-i-check-the-pods-directory-into-source-control
+#
+# Pods/
+#
+# Add this line if you want to avoid checking in source code from the Xcode workspace
+# *.xcworkspace
+
+# Carthage
+#
+# Add this line if you want to avoid checking in source code from Carthage dependencies.
+# Carthage/Checkouts
+
+Carthage/Build/
+
+# fastlane
+#
+# It is recommended to not store the screenshots in the git repo.
+# Instead, use fastlane to re-generate the screenshots whenever they are needed.
+# For more information about the recommended setup visit:
+# https://docs.fastlane.tools/best-practices/source-control/#source-control
+
+fastlane/report.xml
+fastlane/Preview.html
+fastlane/screenshots/**/*.png
+fastlane/test_output

adntmdbapp.xcodeproj/project.xcworkspace/xcuserdata/ask.xcuserdatad/UserInterfaceState.xcuserstate

@@ -28,9 +28,9 @@ struct MovieListView: View {
 
     var body: some View {
         List {
-            ForEach($networkManager.movies) { $movie in
-                NavigationLink(destination: MovieDetailView(networkManager: networkManager, movie: $movie)) {
-                    MovieRowView(movie: $movie, networkManager: networkManager)
+            ForEach(networkManager.movies) { movie in
+                NavigationLink(destination: MovieDetailView(networkManager: networkManager, movie: .constant(movie))) {
+                    MovieRowView(movie: movie, networkManager: networkManager)
                 }
             }
             if networkManager.currentPage <= networkManager.totalPages {
@@ -62,21 +62,30 @@ struct MovieListView: View {
 }
 
 struct MovieRowView: View {
-    @Binding var movie: Movie
+    let movie: Movie
     @ObservedObject var networkManager: NetworkManager
 
     var body: some View {
-        HStack {
+        HStack(spacing: 15) {
             networkManager.posterImage(for: movie)
-                .frame(width: 50, height: 75)
-            VStack(alignment: .leading) {
+                .frame(width: 60, height: 90)
+                .cornerRadius(8)
+            
+            VStack(alignment: .leading, spacing: 5) {
                 Text(movie.title)
                     .font(.headline)
-                Text("Rating: \(String(format: "%.1f", movie.rating))")
-                    .font(.subheadline)
-                    .foregroundColor(.secondary)
+                    .lineLimit(2)
+                
+                HStack {
+                    Image(systemName: "star.fill")
+                        .foregroundColor(.yellow)
+                    Text(String(format: "%.1f", movie.rating))
+                        .font(.subheadline)
+                }
             }
+            
             Spacer()
+            
             Button(action: {
                 networkManager.toggleFavorite(for: movie)
             }) {
@@ -85,6 +94,7 @@ struct MovieRowView: View {
             }
             .buttonStyle(PlainButtonStyle())
         }
+        .padding(.vertical, 8)
     }
 }
 

adntmdbapp/ContentView.swift

@@ -4,15 +4,21 @@ struct FavoriteView: View {
     @ObservedObject var networkManager: NetworkManager
 
     var body: some View {
-        NavigationView {
-            List {
-                ForEach($networkManager.favoriteMovies) { $movie in
-                    NavigationLink(destination: MovieDetailView(networkManager: networkManager, movie: $movie)) {
-                        MovieRowView(movie: $movie, networkManager: networkManager)
-                    }
+        List {
+            ForEach(networkManager.favoriteMovies) { movie in
+                NavigationLink(destination: MovieDetailView(networkManager: networkManager, movie: .constant(movie))) {
+                    MovieRowView(movie: movie, networkManager: networkManager)
                 }
             }
-            .navigationTitle("Favorites")
+            .onDelete(perform: removeItems)
+        }
+        .navigationTitle("Favorites")
+    }
+    
+    func removeItems(at offsets: IndexSet) {
+        for index in offsets {
+            let movie = networkManager.favoriteMovies[index]
+            networkManager.toggleFavorite(for: movie)
         }
     }
 }