fix: remaining spaces in view

pmusolino · pmusolino · commit ee7c6d8cb4d5 · 2022-04-05T15:20:31.000+02:00
diff --git a/WooCommerce/Classes/ViewRelated/Coupons/Add and Edit Coupons/AddEditCoupon.swift b/WooCommerce/Classes/ViewRelated/Coupons/Add and Edit Coupons/AddEditCoupon.swift
@@ -65,6 +65,7 @@ struct AddEditCoupon: View {
                                 .fixedSize()
                                 Spacer().frame(maxWidth: .infinity)
                             }
+                            .padding([.bottom], Constants.verticalSpacing)
 
                             Button {
                                 //TODO: handle action
@@ -83,17 +84,20 @@ struct AddEditCoupon: View {
                                 Divider()
                                     .padding(.leading, Constants.margin)
                             }
+                            .padding([.bottom], Constants.verticalSpacing)
 
                             Group {
                                 TitleAndToggleRow(title: Localization.includeFreeShipping, isOn: .constant(false))
                                     .padding(.horizontal, Constants.margin)
                                 Divider()
                                     .padding(.leading, Constants.margin)
                             }
+                            .padding([.bottom], Constants.verticalSpacing)
                         }
 
                         Group {
                             ListHeaderView(text: Localization.headerApplyCouponTo.uppercased(), alignment: .left)
+                                .padding([.bottom], Constants.verticalSpacing)
 
                             Button {
                                 //TODO: handle action
@@ -106,6 +110,7 @@ struct AddEditCoupon: View {
                             }
                             .buttonStyle(SecondaryButtonStyle())
                             .padding(.horizontal, Constants.margin)
+                            .padding([.bottom], Constants.verticalSpacing)
 
                             Button {
                                 //TODO: handle action
@@ -120,6 +125,7 @@ struct AddEditCoupon: View {
                             .buttonStyle(SecondaryButtonStyle())
                             .padding(.horizontal, Constants.margin)
                         }
+                        .padding([.bottom], Constants.verticalSpacing)
 
                         Group {
                             ListHeaderView(text: Localization.headerUsageDetails.uppercased(), alignment: .left)
@@ -130,6 +136,7 @@ struct AddEditCoupon: View {
                             Divider()
                                 .padding(.leading, Constants.margin)
                         }
+                        .padding([.bottom], Constants.verticalSpacing)
 
                         Button {
                             //TODO: handle action
@@ -138,7 +145,7 @@ struct AddEditCoupon: View {
                         }
                         .buttonStyle(PrimaryButtonStyle())
                         .padding(.horizontal, Constants.margin)
-                        .padding(.top, Constants.verticalSpacing)
+                        .padding([.top, .bottom], Constants.verticalSpacing)
                     }
                 }
                 .ignoresSafeArea(.container, edges: [.horizontal, .bottom])

Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,7 @@ struct AddEditCoupon: View {`
`65`	`65`	`.fixedSize()`
`66`	`66`	`Spacer().frame(maxWidth: .infinity)`
`67`	`67`	`}`
	`68`	`+ .padding([.bottom], Constants.verticalSpacing)`
`68`	`69`
`69`	`70`	`Button {`
`70`	`71`	`//TODO: handle action`
`@@ -83,17 +84,20 @@ struct AddEditCoupon: View {`
`83`	`84`	`Divider()`
`84`	`85`	`.padding(.leading, Constants.margin)`
`85`	`86`	`}`
	`87`	`+ .padding([.bottom], Constants.verticalSpacing)`
`86`	`88`
`87`	`89`	`Group {`
`88`	`90`	`TitleAndToggleRow(title: Localization.includeFreeShipping, isOn: .constant(false))`
`89`	`91`	`.padding(.horizontal, Constants.margin)`
`90`	`92`	`Divider()`
`91`	`93`	`.padding(.leading, Constants.margin)`
`92`	`94`	`}`
	`95`	`+ .padding([.bottom], Constants.verticalSpacing)`
`93`	`96`	`}`
`94`	`97`
`95`	`98`	`Group {`
`96`	`99`	`ListHeaderView(text: Localization.headerApplyCouponTo.uppercased(), alignment: .left)`
	`100`	`+ .padding([.bottom], Constants.verticalSpacing)`
`97`	`101`
`98`	`102`	`Button {`
`99`	`103`	`//TODO: handle action`
`@@ -106,6 +110,7 @@ struct AddEditCoupon: View {`
`106`	`110`	`}`
`107`	`111`	`.buttonStyle(SecondaryButtonStyle())`
`108`	`112`	`.padding(.horizontal, Constants.margin)`
	`113`	`+ .padding([.bottom], Constants.verticalSpacing)`
`109`	`114`
`110`	`115`	`Button {`
`111`	`116`	`//TODO: handle action`
`@@ -120,6 +125,7 @@ struct AddEditCoupon: View {`
`120`	`125`	`.buttonStyle(SecondaryButtonStyle())`
`121`	`126`	`.padding(.horizontal, Constants.margin)`
`122`	`127`	`}`
	`128`	`+ .padding([.bottom], Constants.verticalSpacing)`
`123`	`129`
`124`	`130`	`Group {`
`125`	`131`	`ListHeaderView(text: Localization.headerUsageDetails.uppercased(), alignment: .left)`
`@@ -130,6 +136,7 @@ struct AddEditCoupon: View {`
`130`	`136`	`Divider()`
`131`	`137`	`.padding(.leading, Constants.margin)`
`132`	`138`	`}`
	`139`	`+ .padding([.bottom], Constants.verticalSpacing)`
`133`	`140`
`134`	`141`	`Button {`
`135`	`142`	`//TODO: handle action`
`@@ -138,7 +145,7 @@ struct AddEditCoupon: View {`
`138`	`145`	`}`
`139`	`146`	`.buttonStyle(PrimaryButtonStyle())`
`140`	`147`	`.padding(.horizontal, Constants.margin)`
`141`		`- .padding(.top, Constants.verticalSpacing)`
	`148`	`+ .padding([.top, .bottom], Constants.verticalSpacing)`
`142`	`149`	`}`
`143`	`150`	`}`
`144`	`151`	`.ignoresSafeArea(.container, edges: [.horizontal, .bottom])`