Skip to content

Commit 3a189d2

Browse files
committed
Use navigation stack and tool bar
1 parent 628e7ec commit 3a189d2

File tree

1 file changed

+38
-35
lines changed

1 file changed

+38
-35
lines changed

WooCommerce/Classes/ViewRelated/Orders/Order Details/Shipping Labels/WooShipping Create Shipping Labels/WooShipping Hazmat Section/WooShippingHazmatDetailView.swift

Lines changed: 38 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -12,48 +12,51 @@ struct WooShippingHazmatDetailView: View {
1212
}
1313

1414
var body: some View {
15-
ScrollView {
16-
VStack(spacing: Constants.verticalSpacing) {
17-
HStack {
18-
Button(Localization.cancel) {
19-
dismiss()
20-
}
21-
.padding(.top)
22-
Spacer()
23-
}
15+
NavigationStack {
16+
ScrollView {
17+
VStack(spacing: Constants.verticalSpacing) {
2418

25-
Text(Localization.title)
26-
.secondaryTitleStyle()
27-
.bold()
28-
.frame(maxWidth: .infinity, alignment: .leading)
19+
Text(Localization.title)
20+
.secondaryTitleStyle()
21+
.bold()
22+
.frame(maxWidth: .infinity, alignment: .leading)
2923

30-
Toggle(isOn: $isHazardous) {
31-
Text(Localization.toggleLabel)
32-
}
24+
Toggle(isOn: $isHazardous) {
25+
Text(Localization.toggleLabel)
26+
}
3327

34-
Button(Localization.selectCategory) {
35-
// TODO: navigate to category list
36-
}
37-
.buttonStyle(PrimaryButtonStyle())
38-
.renderedIf(isHazardous)
28+
Button(Localization.selectCategory) {
29+
// TODO: navigate to category list
30+
}
31+
.buttonStyle(PrimaryButtonStyle())
32+
.renderedIf(isHazardous)
3933

40-
Divider()
34+
Divider()
4135

42-
Text(Localization.detailLine1)
43-
.frame(maxWidth: .infinity, alignment: .leading)
44-
Text(detailLine2AttributedString)
45-
.frame(maxWidth: .infinity, alignment: .leading)
46-
Text(detailLine3AttributedString)
47-
.frame(maxWidth: .infinity, alignment: .leading)
36+
Text(Localization.detailLine1)
37+
.frame(maxWidth: .infinity, alignment: .leading)
38+
Text(detailLine2AttributedString)
39+
.frame(maxWidth: .infinity, alignment: .leading)
40+
Text(detailLine3AttributedString)
41+
.frame(maxWidth: .infinity, alignment: .leading)
4842

49-
Spacer()
43+
Spacer()
44+
}
45+
.environment(\.openURL, OpenURLAction { url in
46+
detailURL = url
47+
return .handled
48+
})
49+
.safariSheet(url: $detailURL)
50+
.padding(.horizontal)
51+
.toolbar {
52+
ToolbarItem(placement: .cancellationAction) {
53+
Button(Localization.cancel) {
54+
dismiss()
55+
}
56+
}
57+
}
58+
.toolbarBackground(Color.clear, for: .navigationBar)
5059
}
51-
.environment(\.openURL, OpenURLAction { url in
52-
detailURL = url
53-
return .handled
54-
})
55-
.safariSheet(url: $detailURL)
56-
.padding(.horizontal)
5760
}
5861
}
5962
}

0 commit comments

Comments
 (0)