Skip to content

Commit 7cdb19b

Browse files
committed
Add clearButton method. Add spacing to icon
1 parent c8dde26 commit 7cdb19b

File tree

3 files changed

+18
-7
lines changed

3 files changed

+18
-7
lines changed

WooCommerce/Classes/ViewModels/CardPresentPayments/CardPresentModalScanningForReader.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ final class CardPresentModalScanningForReader: CardPresentPaymentsModalViewModel
77
private let cancelAction: () -> Void
88

99
let textMode: PaymentsModalTextMode = .reducedBottomInfo
10-
let actionsMode: PaymentsModalActionsMode = .secondaryActionAndAuxiliary
10+
let actionsMode: PaymentsModalActionsMode = .secondaryActionAndAttributedAuxiliaryButton
1111

1212
let topTitle: String = Localization.title
1313

WooCommerce/Classes/ViewModels/CardPresentPayments/CardPresentPaymentsModalViewModel.swift

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,8 @@ enum PaymentsModalActionsMode {
8686
/// One secondary action button
8787
case secondaryOnlyAction
8888

89-
/// One secondary action button and an auxiliary button
90-
case secondaryActionAndAuxiliary
89+
/// One secondary action button and an auxiliary button that uses NSAttributedString
90+
case secondaryActionAndAttributedAuxiliaryButton
9191

9292
/// Two action buttons
9393
case twoAction

WooCommerce/Classes/ViewRelated/CardPresentPayments/CardPresentPaymentsModalViewController.swift

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,7 @@ private extension CardPresentPaymentsModalViewController {
158158
}
159159

160160
func styleAuxiliaryButton() {
161-
if viewModel.actionsMode != .secondaryActionAndAuxiliary {
161+
if viewModel.actionsMode != .secondaryActionAndAttributedAuxiliaryButton {
162162
auxiliaryButton.applyLinkButtonStyle()
163163
}
164164
auxiliaryButton.titleLabel?.minimumScaleFactor = 0.5
@@ -279,20 +279,31 @@ private extension CardPresentPaymentsModalViewController {
279279
}
280280

281281
func configureAuxiliaryButton() {
282+
clearAuxiliaryButton()
283+
282284
guard shouldShowAuxiliaryButton() else {
283285
auxiliaryButton.isHidden = true
284286
return
285287
}
286288

287289
auxiliaryButton.isHidden = false
290+
auxiliaryButton.setTitle(viewModel.auxiliaryButtonTitle, for: .normal)
288291
auxiliaryButton.accessibilityIdentifier = Accessibility.auxiliaryButton
289292

290-
if viewModel.actionsMode == .secondaryActionAndAuxiliary {
293+
if viewModel.actionsMode == .secondaryActionAndAttributedAuxiliaryButton {
291294
auxiliaryButton.setImage(.infoOutlineImage, for: .normal)
292295
auxiliaryButton.setAttributedTitle(viewModel.auxiliaryAttributedButtonTitle, for: .normal)
296+
auxiliaryButton.distributeTitleAndImage(spacing: 12.0)
293297
}
294298
}
295299

300+
func clearAuxiliaryButton() {
301+
auxiliaryButton.setImage(nil, for: .normal)
302+
auxiliaryButton.setAttributedTitle(nil, for: .normal)
303+
auxiliaryButton.setTitle(nil, for: .normal)
304+
auxiliaryButton.accessibilityIdentifier = nil
305+
}
306+
296307
func configureSpacer() {
297308
let enabled = !shouldShowActionButtons()
298309

@@ -344,12 +355,12 @@ private extension CardPresentPaymentsModalViewController {
344355
}
345356

346357
func shouldShowBottomActionButton() -> Bool {
347-
[.secondaryOnlyAction, .twoAction, .twoActionAndAuxiliary, .secondaryActionAndAuxiliary]
358+
[.secondaryOnlyAction, .twoAction, .twoActionAndAuxiliary, .secondaryActionAndAttributedAuxiliaryButton]
348359
.contains(viewModel.actionsMode)
349360
}
350361

351362
func shouldShowAuxiliaryButton() -> Bool {
352-
[.twoActionAndAuxiliary, .secondaryActionAndAuxiliary].contains(viewModel.actionsMode)
363+
[.twoActionAndAuxiliary, .secondaryActionAndAttributedAuxiliaryButton].contains(viewModel.actionsMode)
353364
}
354365
}
355366

0 commit comments

Comments
 (0)