PR feedback

Sources/WalletConnectModal/Modal/ModalSheet.swift

@@ -94,6 +94,13 @@ public struct ModalSheet: View {
                         TextField("Search", text: $viewModel.searchTerm, onEditingChanged: { editing in
                             self.searchEditing = editing
                         })
+                        .transform { view in
+                            #if os(macOS)
+                            view
+                            #else
+                            view.autocapitalization(.none)
+                            #endif
+                        }
                     }
                     .padding(.vertical, 4)
                     .padding(.horizontal, 10)
@@ -107,11 +114,6 @@ public struct ModalSheet: View {
                     .onDisappear {
                         searchEditing = false
                     }
-                    .transform {
-                        #if os(iOS)
-                            $0.autocapitalization(.none)
-                        #endif
-                    }
                     .padding(.horizontal, 50)
                 } else {
                     Text(viewModel.destination.contentTitle)

Sources/WalletConnectModal/Modal/ModalViewModel.swift

@@ -79,7 +79,7 @@ final class ModalViewModel: ObservableObject {
             .sink { sessions in
                 print(sessions)
                 isShown.wrappedValue = false
-                self.toast = Toast(style: .success, message: "Session estabilished", duration: 15)
+                self.toast = Toast(style: .success, message: "Session estabilished", duration: 5)
             }
             .store(in: &disposeBag)
 

Sources/WalletConnectModal/Modal/Screens/GetAWalletView.swift

@@ -16,53 +16,54 @@ struct GetAWalletView: View {
                             WalletImage(wallet: wallet)
                                 .frame(width: 40, height: 40)
 
-                        Text(wallet.name)
-                            .font(.system(size: 16, weight: .medium))
-                            .padding(.horizontal)
+                            Text(wallet.name)
+                                .font(.system(size: 16, weight: .medium))
+                                .padding(.horizontal)
 
-                        Spacer()
+                            Spacer()
 
-                        Image(systemName: "chevron.right")
-                            .font(.system(.footnote).weight(.semibold))
+                            Image(systemName: "chevron.right")
+                                .font(.system(.footnote).weight(.semibold))
+                        }
                     }
                 }
             }
-        }
-        .frame(minHeight: 400)
-        .listStyle(.plain)
+            .frame(minHeight: 400)
+            .listStyle(.plain)
 
 
             VStack(alignment: .center, spacing: 8) {
                 Text("Not what you’re looking for?")
                     .font(
                         .system(size: 16)
                         .weight(.semibold)
-                )
-                .multilineTextAlignment(.center)
-                .foregroundColor(.foreground1)
+                    )
+                    .multilineTextAlignment(.center)
+                    .foregroundColor(.foreground1)
 
-            Text("With hundreds of wallets out there, 
there’s something for everyone ")
-                .font(
-                    .system(size: 14)
+                Text("With hundreds of wallets out there, 
there’s something for everyone ")
+                    .font(
+                        .system(size: 14)
                         .weight(.medium)
-                )
-                .foregroundColor(.foreground2)
-                .fixedSize(horizontal: false, vertical: true)
+                    )
+                    .foregroundColor(.foreground2)
+                    .fixedSize(horizontal: false, vertical: true)
 
-            Button(action: {
-                navigateToExternalLink(URL(string: "https://walletconnect.com/explorer?type=wallet")!)
-            }) {
-                HStack {
-                    Text("Explore Wallets")
-                    Image(.external_link)
+                Button(action: {
+                    navigateToExternalLink(URL(string: "https://walletconnect.com/explorer?type=wallet")!)
+                }) {
+                    HStack {
+                        Text("Explore Wallets")
+                        Image(.external_link)
+                    }
                 }
+                .buttonStyle(W3MButtonStyle())
             }
-            .buttonStyle(W3MButtonStyle())
+            .multilineTextAlignment(.center)
+            .padding(.horizontal, 0)
+            .padding(.top, 0)
+            .frame(maxHeight: .infinity, alignment: .bottom)
         }
-        .multilineTextAlignment(.center)
-        .padding(.horizontal, 0)
-        .padding(.top, 0)
-        .frame(maxHeight: .infinity, alignment: .bottom)
     }
 }
 

Sources/WalletConnectModal/Modal/Screens/WhatIsWalletView.swift

@@ -23,7 +23,11 @@ struct WhatIsWalletView: View {
                             .padding(.bottom, 40)
                     }
                 }
-                .tabViewStyle(.page(indexDisplayMode: .always))
+                .transform {
+                    #if os(iOS)
+                    $0.tabViewStyle(.page(indexDisplayMode: .always))
+                    #endif
+                }
                 .frame(height: 150)
 
             } else {

Sources/WalletConnectModal/UI/Common/Toast.swift

@@ -1,8 +1,8 @@
 import SwiftUI
 
 struct Toast: Equatable {
-    var style: ToastStyle
-    var message: String
+    let style: ToastStyle
+    let message: String
     var duration: Double = 3
     var width: Double = .infinity
 }