Skip to content

Commit 149a78c

Browse files
authored
Merge pull request #7334 from woocommerce/issue/7305-cancel-button-feature-announcement-alert
Add cancel button to the Upsell Card Reader banner dismiss dialog
2 parents b6348ad + d6f312a commit 149a78c

File tree

1 file changed

+21
-15
lines changed

1 file changed

+21
-15
lines changed

WooCommerce/Classes/ViewRelated/ReusableViews/SwiftUI Components/FeatureAnnouncementCardView.swift

Lines changed: 21 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import SwiftUI
22

33
struct FeatureAnnouncementCardView: View {
44
private let viewModel: FeatureAnnouncementCardViewModel
5-
@State private var showingDismissAlert = false
5+
@State private var showingDismissActionSheet = false
66

77
let dismiss: (() -> Void)?
88
let callToAction: (() -> Void)?
@@ -23,25 +23,31 @@ struct FeatureAnnouncementCardView: View {
2323
Spacer()
2424
if let dismiss = dismiss {
2525
Button(action: {
26-
showingDismissAlert = true
26+
showingDismissActionSheet = true
2727
}) {
2828
Image(systemName: "xmark")
2929
.foregroundColor(Color(.withColorStudio(.gray)))
3030
}
3131
.padding(.trailing, Layout.padding)
32-
.alert(isPresented: $showingDismissAlert,
33-
content: {
34-
Alert(title: Text(viewModel.dismissAlertTitle),
35-
message: Text(viewModel.dismissAlertMessage),
36-
primaryButton: .cancel(Text(Localization.remindLaterButton), action: {
37-
viewModel.remindLaterTapped()
38-
dismiss()
39-
}),
40-
secondaryButton: .default(Text(Localization.dontShowAgainButton), action: {
41-
viewModel.dontShowAgainTapped()
42-
dismiss()
43-
}))
44-
})
32+
.actionSheet(isPresented: $showingDismissActionSheet) {
33+
ActionSheet(
34+
title: Text(viewModel.dismissAlertTitle),
35+
message: Text(viewModel.dismissAlertMessage),
36+
buttons: [
37+
.default(Text(Localization.remindLaterButton),
38+
action: {
39+
viewModel.remindLaterTapped()
40+
dismiss()
41+
}),
42+
.default(Text(Localization.dontShowAgainButton),
43+
action: {
44+
viewModel.dontShowAgainTapped()
45+
dismiss()
46+
}),
47+
.cancel()
48+
]
49+
)
50+
}
4551
}
4652
}
4753
.padding(.top, Layout.padding)

0 commit comments

Comments
 (0)