From 7d485f5937898ff0ad84025363817102ad8e4b82 Mon Sep 17 00:00:00 2001
From: Bill Richardson <william.richardson@safelite.com>
Date: Thu, 1 Aug 2024 13:23:41 -0400
Subject: [PATCH 1/2] service package updates

rearrange package order
add red border on continue error when nothing selected
collapse on mobile
---
 .../service-package-question.vue              | 10 ++--
 .../service-package-radio.vue                 | 55 ++++++++++++-------
 src/styles/common-error-styles.scss           | 13 +++++
 3 files changed, 54 insertions(+), 24 deletions(-)

diff --git a/src/layouts/service-packages/service-package-question/service-package-question.vue b/src/layouts/service-packages/service-package-question/service-package-question.vue
index bad1f443..87fa7bbd 100644
--- a/src/layouts/service-packages/service-package-question/service-package-question.vue
+++ b/src/layouts/service-packages/service-package-question/service-package-question.vue
@@ -50,7 +50,7 @@ export default {
     data() {
         return {
             servicePackageRadio,
-            selectedPackageName: packageNames.TIER_ONE
+            selectedPackageName: ''
         };
     },
     computed: {
@@ -64,10 +64,6 @@ export default {
         servicePackageAnswers() {
             if (!this.cmsWidgetName) return [];
             const cmsAnswersContent = [];
-            cmsAnswersContent.push({
-                Name: 'TierOne',
-                cmsWidgetName: 'EconomyServicePackage'
-            });
             if (this.shouldDisplayTierTwoPackage) {
                 cmsAnswersContent.push({
                     Name: 'TierTwo',
@@ -78,6 +74,10 @@ export default {
                 Name: 'TierThree',
                 cmsWidgetName: 'PremiumServicePackage'
             });
+            cmsAnswersContent.push({
+                Name: 'TierOne',
+                cmsWidgetName: 'EconomyServicePackage'
+            });
             // if cms content has not yet loaded, skip
             if (!this.getCmsContent(cmsAnswersContent[0].cmsWidgetName, 'HeaderText')
                 || this.getCmsContent(cmsAnswersContent[0].cmsWidgetName, 'HeaderText') === '') {
diff --git a/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue b/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
index 5465bad6..83426fd7 100644
--- a/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
+++ b/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
@@ -7,17 +7,20 @@
             :class="[buttonLabelSubCopy ? 'has-subheader' : '']"
             for="testradio">
             <div class="package-specs">
-                <p class="m-0">
-                    <span v-html="buttonLabel"></span>
-                    <span
-                        class="pricing-info"
-                        v-html="buttonAuxiliaryCopy"></span>
-                </p>
-                <p
-                   v-if="buttonLabelSubCopy"
-                   class="sub-label m-0"
-                   v-html="buttonLabelSubCopy"></p>
                 <div>
+                    <p class="m-0">
+                        <span v-html="buttonLabel"></span>
+                        <span
+                            class="pricing-info"
+                            v-html="buttonAuxiliaryCopy"></span>
+                    </p>
+                    <p
+                        v-if="buttonLabelSubCopy"
+                        class="sub-label m-0"
+                        v-html="buttonLabelSubCopy">
+                    </p>
+                </div>
+                <div class="hide-when-closed">
                     <ul>
                         <li
                             v-for="listItem in arrayOfListItemsFromBodyText"
@@ -139,6 +142,10 @@ export default {
                     border-radius: 0.5rem;
                     overflow: hidden;
                     min-height: 60px;
+                    max-height: 100px;
+                    @include media-breakpoint-up(md) {
+                        max-height: 500px;
+                    }
 
                     &.has-subheader {
                         min-height: 80px;
@@ -222,19 +229,21 @@ export default {
 
             .package-footer {
                 color: $red;
-                font-weight: $font-weight-bold;
-                margin-top: 0.5rem;
+                font-weight: $font-weight-bold; // 600 in fmg
+                margin-top: 0.5rem; // not in fmg
             }
 
             .package-specs {
                 display: flex;
                 flex-direction: column;
                 width: 100%;
-                max-height: 1000px;
-                transition: all 0.5s ease;
-
+                max-height: 0;
+                transition: all 1s ease;
+                @include media-breakpoint-up(md) {
+                    max-height: 500px;
+                }
                 p {
-                    font-weight: 500;
+                    font-weight: $font-weight-bold; // 600 in fmg
                     display: flex;
                     justify-content: space-between;
 
@@ -253,16 +262,16 @@ export default {
                 }
 
                 ul {
-                    margin: 1rem 0 0 -.6rem;
+                    margin: 1rem 0 0 -.6rem; // .9375rem in fmg
                     padding: 0;
 
                     li {
                         margin-bottom: 0.5rem;
                         font-size: 0.875rem;
-                        line-height: 1.714;
+                        line-height: 1.5rem;
 
                         a {
-                            line-height: 1.714;
+                            line-height: 1.5rem;
                             padding: 0;
                         }
                     }
@@ -278,6 +287,14 @@ export default {
                         opacity: 0;
                     }
                 }
+
+                .hide-when-closed {
+                    display: none;
+                    @include media-breakpoint-up(md) {
+                        display: flex;
+                        flex-direction: column;
+                    }
+                }
             }
         }
     }
diff --git a/src/styles/common-error-styles.scss b/src/styles/common-error-styles.scss
index ed69b1fd..7df67aec 100644
--- a/src/styles/common-error-styles.scss
+++ b/src/styles/common-error-styles.scss
@@ -136,6 +136,19 @@ html {
         }
     }
 
+    //Quote page radio button group
+    .package-main {
+        .package-wrapper {
+            .has-error {
+                input[type="radio"] {
+                    +.package-label {
+                        border: 1px solid $red;
+                    }
+                }
+            }
+        }
+    }
+
     //Restore to default style if alert box is present
     .alertError {
         .has-error {

From 456cdc9c479889b039a3ff893a089aff1c2c86f4 Mon Sep 17 00:00:00 2001
From: Bill Richardson <william.richardson@safelite.com>
Date: Thu, 1 Aug 2024 14:08:19 -0400
Subject: [PATCH 2/2] update min size with sub text

given current css, needs to be 86px to prevent a small blip of sizing when going from standard to another choice.
---
 .../service-package-radio/service-package-radio.vue             | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue b/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
index 83426fd7..3a035bf4 100644
--- a/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
+++ b/src/layouts/service-packages/service-package-question/service-package-radio/service-package-radio.vue
@@ -148,7 +148,7 @@ export default {
                     }
 
                     &.has-subheader {
-                        min-height: 80px;
+                        min-height: 86px;
                     }
 
                     &:before {