Skip to content

Commit 952ba9e

Browse files
committed
Use navigationDestination API for product selector view presenting
1 parent 56e1a54 commit 952ba9e

File tree

1 file changed

+17
-11
lines changed

1 file changed

+17
-11
lines changed

WooCommerce/Classes/ViewRelated/Products/ProductSelector/ProductSelectorView.swift

Lines changed: 17 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -116,17 +116,23 @@ struct ProductSelectorView: View {
116116
.padding(Constants.defaultPadding)
117117
.accessibilityIdentifier(Constants.doneButtonAccessibilityIdentifier)
118118
.renderedIf(configuration.multipleSelectionEnabled && viewModel.syncApproach == .onButtonTap)
119-
120-
if let variationListViewModel = viewModel.productVariationListViewModel {
121-
LazyNavigationLink(destination: ProductVariationSelectorView(
122-
isPresented: $isPresented,
123-
viewModel: variationListViewModel,
124-
onMultipleSelections: { selectedIDs in
125-
viewModel.updateSelectedVariations(productID: variationListViewModel.productID, selectedVariationIDs: selectedIDs)
126-
}), isActive: $viewModel.isShowingProductVariationList) {
127-
EmptyView()
128-
}
129-
.renderedIf(configuration.treatsAllProductsAsSimple == false)
119+
}
120+
.if(configuration.treatsAllProductsAsSimple == false) { view in
121+
view.navigationDestination(isPresented: $viewModel.isShowingProductVariationList) {
122+
if let variationListViewModel = viewModel.productVariationListViewModel {
123+
ProductVariationSelectorView(
124+
isPresented: $isPresented,
125+
viewModel: variationListViewModel,
126+
onMultipleSelections: { selectedIDs in
127+
viewModel.updateSelectedVariations(
128+
productID: variationListViewModel.productID,
129+
selectedVariationIDs: selectedIDs
130+
)
131+
}
132+
)
133+
} else {
134+
EmptyView()
135+
}
130136
}
131137
}
132138
.padding(.horizontal, insets: safeAreaInsets)

0 commit comments

Comments
 (0)