Merge pull request #2150 from Safelite/CSR-2417-close-cart-on-submit

CSR-2417 close cart on submit

src/fmg-components/cart/cart.vue

@@ -199,6 +199,7 @@ export default {
         shouldHideRecalibration: Boolean,
         isItac: Boolean,
         isNoComp: Boolean,
+        isExpandedOnLoad: Boolean,
     },
     data() {
         return {
@@ -217,7 +218,7 @@ export default {
             this.isExpanded = !this.isExpanded;
         },
         updateIsExpandedWithDefault() {
-            this.isExpanded = this.shouldHideRecalibration;
+            this.isExpanded = this.isExpandedOnLoad;
         },
         getVapsPrice(packageName) {
             const vapsItems = this.getVapsCartItemsForSelectedPackage(packageName);
@@ -331,9 +332,8 @@ export default {
     },
     computed: {
         isCartReadyToLoad() {
-            var myReturn = this.shouldHideRecalibration !== null;
             this.updateIsExpandedWithDefault();
-            return myReturn;
+            return this.isExpandedOnLoad !== null;
         },
         lineItems: {
             get: function () {

src/layouts/confirmation/confirmation.vue

@@ -60,6 +60,7 @@
                                 :insuranceCompanyName="insuranceCompanyName"
                                 :showInsuranceCoverageAs="showInsuranceCoverageAs"
                                 :shouldHideRecalibration="shouldHideRecalibration"
+                                :isExpandedOnLoad="false"
                                 :isItac="isItac"
                                 :isNoComp="isNoComp" />
 

src/layouts/payment-method/payment-method.vue

@@ -31,6 +31,7 @@
                         :insuranceCompanyName="insuranceCompanyName"
                         :showInsuranceCoverageAs="showInsuranceCoverageAs"
                         :shouldHideRecalibration="shouldHideRecalibration"
+                        :isExpandedOnLoad="false"
                         :isItac="isItac"
                         :isNoComp="isNoComp" />
 

src/layouts/payment/payment.vue

@@ -59,7 +59,8 @@
                         :insuranceCompanyName="insuranceCompanyName"
                         :showInsuranceCoverageAs="showInsuranceCoverageAs"
                         :isItac="isItac"
-                        :isNocomp="isNocomp" />
+                        :isNocomp="isNocomp"
+                        :isExpandedOnLoad="true" />
 
                     <hr class="mb-5" />
 

src/layouts/quote/quote.vue

@@ -429,7 +429,11 @@ export default {
         },
         shouldHideRecalibration() {
             if (this.isInsuranceSelected) return false;
-            return (experimentMixin.methods.getSettingValue(experimentSettings.RECAL_PRICE_REMOVE)?.toLowerCase() === "true" && this.isRecalibrationOnOrder);
+            return (
+                experimentMixin.methods
+                    .getSettingValue(experimentSettings.RECAL_PRICE_REMOVE)
+                    ?.toLowerCase() === "true" && this.isRecalibrationOnOrder
+            );
         },
         showAfterpayBanner() {
             return (

src/store/index.js

@@ -41,7 +41,11 @@ import {
     coverageTypeValue,
     coverageTypeEnum,
 } from "@/constants/insurance";
-import { containsRecalParts, getTopLevelPartsWithRecal } from "@/helpers/recal-helper";
+import {
+    containsRecalParts,
+    getTopLevelPartsWithRecal,
+    isRecalPartOrHasChildRecalPart,
+} from "@/helpers/recal-helper";
 import { externalParameterStatus } from "@/constants/external-parameters";
 import { experimentSettings } from "@/constants/experiments";
 import experimentMixin from "@/mixins/experiment-mixin.js";
@@ -1651,9 +1655,10 @@ export const actions = {
         const partialLineItemsObjects = context.getters.order.lineItems.glassParts?.map(
             (lineItem) => ({
                 partNumber: lineItem.partNumber,
-                recalibrationType: lineItem.recalibrationType
+                recalibrationType:
-                    ? escapeRecalibrationType(lineItem.recalibrationType)
+                    lineItem.recalibrationType && isRecalPartOrHasChildRecalPart(lineItem)
-                    : undefined,
+                        ? escapeRecalibrationType(lineItem.recalibrationType)
+                        : undefined,
             })
         );
 

src/styles/common-styles.scss

@@ -71,11 +71,7 @@ body {
 
     // Fix "iOS viewport scroll bug" issue on iPhone
     // where bottom of page is covered by address bar
-    @media only screen 
+    @media only screen and (min-device-width: 375px) and (max-device-width: 812px) and (-webkit-min-device-pixel-ratio: 3) and (orientation: portrait) {
-        and (min-device-width: 375px) 
-        and (max-device-width: 812px) 
-        and (-webkit-min-device-pixel-ratio: 3)
-        and (orientation: portrait) { 
         .page-container-grouped-styles {
             padding-bottom: 3rem;
         }