diff --git a/TalerWallet1/Views/Actions/Peer2peer/P2PReadyV.swift b/TalerWallet1/Views/Actions/Peer2peer/P2PReadyV.swift index 98ec9861..5f28b67a 100755 --- a/TalerWallet1/Views/Actions/Peer2peer/P2PReadyV.swift +++ b/TalerWallet1/Views/Actions/Peer2peer/P2PReadyV.swift @@ -86,14 +86,32 @@ struct P2PReadyV: View { let _ = Self._printChanges() let _ = symLog.vlog() // just to get the # to compare it with .onAppear & onDisappear #endif - let navTitle = String(localized: "Ready") + // No hardcoded title: the summary derives one from the + // transaction itself (`navTitleStr` -> `talerTX.localizedType`), + // which is what the transactions list already passes `nil` to get. + // + // Hardcoding "Ready" said the same thing twice. The detail card + // shows the transaction's state on its right, and for a pending + // peer transfer that state IS `ready` -- `MinorState.ready`, + // "Ready" (Transaction.swift:79) -- so the screen read + // + // staticText 'Ready' [16, 64, 100, 41] the title + // staticText 'Ready' [307, 154, 79, 38] the state + // + // captured in `iq3-e00000` seq 35 and 36. The state line shows the + // minor state when `developerMode && pending`, and `developerMode` + // defaults to true in a DEBUG build + // (ThreeAmountsSection.swift:25-29), so that is what a tester sees. + // + // Entering the same transaction from the list instead shows + // "Send Money", which says something the state line does not. ZStack { if let transactionId { TransactionSummaryList(stack: stack.push(), // scope: scope, transactionId: transactionId, talerTX: $talerTX, - navTitle: navTitle, + navTitle: nil, hasDone: true, // conclude p2p showDone: .prominent, url: nil,