From a88e0108c60e5ec06b55e0c549ded7ff74af8ee1 Mon Sep 17 00:00:00 2001
From: Bryan Mauger <bryan.mauger@safelite.com>
Date: Wed, 27 Mar 2024 10:16:03 -0400
Subject: [PATCH 1/3] Update navigation links to open in new window.

---
 .../funnel-header/menu-modal/menu-modal.vue   | 11 +++---
 src/ux-components/text-link/text-link.vue     | 34 +++++++++----------
 2 files changed, 21 insertions(+), 24 deletions(-)

diff --git a/src/fmg-components/funnel-header/menu-modal/menu-modal.vue b/src/fmg-components/funnel-header/menu-modal/menu-modal.vue
index dd77ea7fa..be2cdab23 100644
--- a/src/fmg-components/funnel-header/menu-modal/menu-modal.vue
+++ b/src/fmg-components/funnel-header/menu-modal/menu-modal.vue
@@ -41,12 +41,12 @@
                 </div>
                 <div class="modal-body d-flex flex-column">
                     <textLink
-                        linkType="navigation"
+                        linkType="newWindowLink"
                         text="Terms of use"
                         href="//www.safelite.com/terms-of-use"
                         target="_blank" />
                     <textLink
-                        linkType="navigation"
+                        linkType="newWindowLink"
                         text="Your privacy choices"
                         href="//www.safelite.com/privacy-center"
                         target="_blank">
@@ -58,12 +58,12 @@
                         </template>
                     </textLink>
                     <textLink
-                        linkType="navigation"
+                        linkType="newWindowLink"
                         text="Warranty"
                         href="//www.safelite.com/national-lifetime-warranty"
                         target="_blank" />
                     <textLink
-                        linkType="navigation"
+                        linkType="newWindowLink"
                         text="Notice at collection"
                         href="https://www.safelite.com/ccpa-privacy-policy"
                         target="_blank" />
@@ -177,9 +177,6 @@ export default {
         .modal-fullscreen {
             width: 100vw;
         }
-        .navigation-link {
-            margin-bottom: 1.5rem;
-        }
         .modal-footer {
             border-top: none;
             padding: 2rem;
diff --git a/src/ux-components/text-link/text-link.vue b/src/ux-components/text-link/text-link.vue
index 0490ccc5a..34893da77 100644
--- a/src/ux-components/text-link/text-link.vue
+++ b/src/ux-components/text-link/text-link.vue
@@ -1,20 +1,28 @@
 <template>
     <loadingModal v-if="useLoadingModal" notFullScreen ref="loadingModal" />
-    <a v-if="linkType === 'navigation'" @click="handleClick" class="navigation-link" :href="href"
+    <a v-if="linkType === 'navigation'"
+        @click="handleClick"
+        class="navigation-link"
+        :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
     <a
-        v-else-if="linkType === 'footer'"
+        v-else-if="linkType === 'newWindowLink'"
         @click="handleClick"
-        class="footer-link"
+        class="new-window-link"
         :href="href"
         target="_blank"
         >{{ text }}<slot name="after-text"></slot
     ></a>
-    <a v-else-if="linkType === 'textSmall'" @click="handleClick" class="small" :href="href"
+    <a v-else-if="linkType === 'textSmall'"
+        @click="handleClick"
+        class="small"
+        :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
-    <a v-else-if="linkType === 'text'" @click="handleClick" :href="href"
+    <a v-else-if="linkType === 'text'"
+        @click="handleClick"
+        :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
 </template>
@@ -74,23 +82,15 @@ a {
             color: $blue-700;
         }
     }
-    &.navigation-link {
+    &.navigation-link,
+    &.new-window-link {
         color: $black;
         line-height: 26px;
         white-space: nowrap;
         align-items: center;
     }
-    &.footer-link {
-        color: $gray-600;
-        text-decoration: none;
-        line-height: 20px;
-        padding: 0 0 2px 0;
-        font-weight: 400;
-        font-size: 0.75rem;
-        &:hover {
-            padding: 0 0 2px 0;
-            text-decoration: underline;
-        }
+    &.new-window-link {
+        margin-bottom: 1.5rem;
     }
 }
 </style>

From 85fba15548ef611153d7c46cbfb3608ac119ffc0 Mon Sep 17 00:00:00 2001
From: Bryan Mauger <bryan.mauger@safelite.com>
Date: Wed, 27 Mar 2024 10:21:36 -0400
Subject: [PATCH 2/3] Format code.

---
 src/ux-components/text-link/text-link.vue | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)

diff --git a/src/ux-components/text-link/text-link.vue b/src/ux-components/text-link/text-link.vue
index 34893da77..7fc4eaef8 100644
--- a/src/ux-components/text-link/text-link.vue
+++ b/src/ux-components/text-link/text-link.vue
@@ -1,9 +1,6 @@
 <template>
     <loadingModal v-if="useLoadingModal" notFullScreen ref="loadingModal" />
-    <a v-if="linkType === 'navigation'"
-        @click="handleClick"
-        class="navigation-link"
-        :href="href"
+    <a v-if="linkType === 'navigation'" @click="handleClick" class="navigation-link" :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
     <a
@@ -14,15 +11,10 @@
         target="_blank"
         >{{ text }}<slot name="after-text"></slot
     ></a>
-    <a v-else-if="linkType === 'textSmall'"
-        @click="handleClick"
-        class="small"
-        :href="href"
+    <a v-else-if="linkType === 'textSmall'" @click="handleClick" class="small" :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
-    <a v-else-if="linkType === 'text'"
-        @click="handleClick"
-        :href="href"
+    <a v-else-if="linkType === 'text'" @click="handleClick" :href="href"
         >{{ text }}<slot name="after-text"></slot
     ></a>
 </template>

From 5e04401720930c84f491c47141d6a0c0c4802e8c Mon Sep 17 00:00:00 2001
From: Bryan Mauger <bryan.mauger@safelite.com>
Date: Wed, 27 Mar 2024 10:36:02 -0400
Subject: [PATCH 3/3] Update unit test.

---
 src/ux-components/text-link/text-link.spec.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/ux-components/text-link/text-link.spec.js b/src/ux-components/text-link/text-link.spec.js
index 8b48ad5eb..d26c1a17b 100644
--- a/src/ux-components/text-link/text-link.spec.js
+++ b/src/ux-components/text-link/text-link.spec.js
@@ -18,11 +18,11 @@ describe("text-link.vue", () => {
         expect(paragraph.attributes("class")).toContain("navigation-link");
     });
 
-    it("Should return class footer", async () => {
+    it("Should return class new-window-link", async () => {
         // Act
         const wrapper = shallowMount(textLink, {
             propsData: {
-                linkType: "footer",
+                linkType: "newWindowLink",
             },
         });
 
@@ -30,7 +30,7 @@ describe("text-link.vue", () => {
         const paragraph = wrapper.find("a");
 
         // Expect
-        expect(paragraph.attributes("class")).toContain("footer");
+        expect(paragraph.attributes("class")).toContain("new-window-link");
     });
 
     it("Should return class text-small", async () => {