From 30dfabf8e703802e0f6c6dea418b74b137acc2ca Mon Sep 17 00:00:00 2001 From: cdricms <36056008+cdricms@users.noreply.github.com> Date: Fri, 24 Jul 2026 14:46:44 +0200 Subject: [PATCH] refactor(apple): move share to the toolbar and delete to the bottom Put a share icon in the transfer-details toolbar (opening the QR/share panel) in place of the delete button, drop the now-redundant Share row from the list, and move Delete transfer into the bottom section alongside Stop sharing. --- .../Features/Send/TransferDetailsView.swift | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/apple/VniDrop/Features/Send/TransferDetailsView.swift b/apple/VniDrop/Features/Send/TransferDetailsView.swift index 71e57ad..da537d6 100644 --- a/apple/VniDrop/Features/Send/TransferDetailsView.swift +++ b/apple/VniDrop/Features/Send/TransferDetailsView.swift @@ -44,22 +44,19 @@ struct TransferDetailsView: View { count: pendingReceivers + completedReceivers, onTap: model.openReceivers ) - DetailDestination( - title: String(localized: L10n.Transfer.shareTitle), - description: String(localized: L10n.Transfer.shareDescription), - count: 0, - onTap: model.openShare - ) } - if isActiveShare { - Section { + Section { + if isActiveShare { Button(role: .destructive) { showStopConfirmation = true } label: { Label(String(localized: L10n.Send.stopSharing), systemSymbol: .stopCircle) } } + Button(role: .destructive, action: model.requestDeleteTransfer) { + Label(String(localized: L10n.Button.deleteTransfer), systemSymbol: .trash) + } } } .formStyle(.grouped) @@ -69,9 +66,10 @@ struct TransferDetailsView: View { #endif .toolbar { ToolbarItem(placement: .primaryAction) { - Button(role: .destructive, action: model.requestDeleteTransfer) { - Image(systemSymbol: .trash) + Button(action: model.openShare) { + Label(String(localized: L10n.Transfer.shareTitle), systemSymbol: .squareAndArrowUp) } + .help(String(localized: L10n.Transfer.shareTitle)) } } .confirmationDialog(