[Woo POS] Add totals view. Shuffle views on checkout flow #12809
Conversation
| .padding(.all, 10) | ||
| .frame(maxWidth: .infinity, idealHeight: 120) | ||
| .font(.title) | ||
| .foregroundColor(Color.primaryBackground) | ||
| .background(Color.white) | ||
| Button("Card") { debugPrint("Not implemented") } | ||
| .padding(.all, 10) | ||
| .frame(maxWidth: .infinity, idealHeight: 120) | ||
| .font(.title) | ||
| .foregroundColor(Color.primaryBackground) | ||
| .background(Color.white) |
There was a problem hiding this comment.
Since most likely we'll share button styling in the POS, even if is not final, I think we can start to extract those and add more ButtonStyles to /ReusableViews?
|
|
| Text("Totals") | ||
| .font(.title) | ||
| .foregroundColor(Color.white) | ||
| ForEach(viewModel.productsInCart, id: \.product.productID) { cartProduct in |
There was a problem hiding this comment.
Yup yup! 6997ec8
| .background(Color.secondaryBackground) | ||
| .frame(maxWidth: .infinity) | ||
| Spacer() | ||
| CartView(viewModel: viewModel) |
There was a problem hiding this comment.
you can extract this CartView and its setup into private var:
private var cartView: some View {
CartView(viewModel: viewModel)
.background(Color.secondaryBackground)
.frame(maxWidth: .infinity)
}
You can do same thing with TotalsView and ProductGridView and just use those vars here in switch.
There was a problem hiding this comment.
Sounds good, I've moved all subviews to variables on e5b9f11
| .foregroundColor(.white) | ||
| } | ||
| Spacer() | ||
| HStack { |
There was a problem hiding this comment.
You can put padding to this HStack, or some other layout tweak, so that button background (white) does not overflow to the bottom:
There was a problem hiding this comment.
Good catch, this only happens in landscape so I missed it. Fixed on 4896b7f
…4-pos-checkout-view-shuffle
Partially addresses #12763
Closes: #12764
This PR adds a initial
Totalsview, and shuffles the order of the different views depending if we'rebuildingorfinalizingcheckout. Most likely the view style will change, so I haven't taken much time with the UI details (eg: "Take payment" most likely will be removed or not be a button)Description
Changes
TotalsView. At the moment this just renders a list of products in-cart, and three buttons at the bottom for payment collectionOrderStageenum that indicates where in the process are we,buildingorfinalizing.Testing instructions
CheckoutAdd Morebuildingand cart state is updated