From b4585b368f9daf1cfb513aa369ddfc4a655a7639 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 15:49:39 -0500
Subject: [PATCH 1/8] PXBF-2016-design-qa-tweaks: hide expand all button on
 mobile

---
 .../ResultsView/components/Results/_index.scss     | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
index 5e1a7e98..08c69a62 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
@@ -4,9 +4,9 @@
 @use '@styles/functions' as *;
 
 .bf-result-view {
- .bf-grid-container.grid-container {
-  padding: rem(20px);
- }
+  .bf-grid-container.grid-container {
+    padding: rem(20px);
+  }
 }
 
 .bf-result-view-benefits,
@@ -21,6 +21,10 @@
   .bf-result-view-share-results-heading {
     @include h3;
   }
+
+  .bf-expand-all {
+    display: none;
+  }
 }
 
 .bf-result-view-unmet {
@@ -48,4 +52,8 @@
   .bf-result-view-details {
     padding: rem(24px) 5.4rem;
   }
+
+  .bf-result-view-benefits .bf-expand-all {
+    display: block;
+  }
 }

From d67085ba26bfea87e6fa5a57e928ff3435d19f5a Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 15:50:44 -0500
Subject: [PATCH 2/8] PXBF-2016-design-qa-tweaks: update content for cta

---
 .../ResultsView/__tests__/__snapshots__/index.spec.jsx.snap   | 4 ++--
 benefit-finder/src/shared/locales/en/en.json                  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
index 6a1bd88f..3625590d 100644
--- a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
@@ -6215,7 +6215,7 @@ exports[`loads view 1`] = `
                 data-testid="bf-result-view-unmet-button"
                 type="button"
               >
-                Explore benefits you did not qualify for
+                See benefits you didn't qualify for
               </button>
             </div>
           </div>
@@ -12554,7 +12554,7 @@ exports[`scenario 1 loads in view with the correct amount of likely eligible ite
                 data-testid="bf-result-view-unmet-button"
                 type="button"
               >
-                Explore benefits you did not qualify for
+                See benefits you didn't qualify for
               </button>
             </div>
           </div>
diff --git a/benefit-finder/src/shared/locales/en/en.json b/benefit-finder/src/shared/locales/en/en.json
index 6010535f..584b6f9d 100644
--- a/benefit-finder/src/shared/locales/en/en.json
+++ b/benefit-finder/src/shared/locales/en/en.json
@@ -153,7 +153,7 @@
     "notEligibleResults": {
       "heading": "Benefits you did not qualify for",
       "description": "<div>Based on your answers you are not eligible for these benefits. <strong>You may become eligible if you enter additional information or your situation changes.</strong></div>",
-      "cta": "Explore benefits you did not qualify for"
+      "cta": "See benefits you didn't qualify for"
     },
     "resultsRelativeBenefits": {
       "heading": "More benefits"

From 83ddf93b412fb1c2888e99dc5a12d17e66441ae5 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:02:00 -0500
Subject: [PATCH 3/8] PXBF-2016-design-qa-tweaks: bold content and include
 createMarkup

---
 .../__tests__/__snapshots__/index.spec.jsx.snap    | 14 ++++++++++++--
 .../__tests__/__snapshots__/index.spec.jsx.snap    |  7 ++++++-
 .../ResultsView/components/Results/index.jsx       |  2 +-
 benefit-finder/src/shared/locales/en/en.json       |  2 +-
 benefit-finder/src/shared/locales/es/es.json       |  2 +-
 5 files changed, 21 insertions(+), 6 deletions(-)

diff --git a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
index 3625590d..9595482c 100644
--- a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
@@ -6303,7 +6303,12 @@ exports[`loads view 1`] = `
               Share results
             </h3>
             <p>
-              Copy or email these results. Your answers will be visible. Only share with those you trust.
+              Copy or email these results. Your answers will be visible. 
+              <strong>
+                Only share with those you trust
+                <!--&lt;strong-->
+                .
+              </strong>
             </p>
             <ul
               class="bf-result-view-share-results-button-group"
@@ -12642,7 +12647,12 @@ exports[`scenario 1 loads in view with the correct amount of likely eligible ite
               Share results
             </h3>
             <p>
-              Copy or email these results. Your answers will be visible. Only share with those you trust.
+              Copy or email these results. Your answers will be visible. 
+              <strong>
+                Only share with those you trust
+                <!--&lt;strong-->
+                .
+              </strong>
             </p>
             <ul
               class="bf-result-view-share-results-button-group"
diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
index cbb95b95..9b0726c2 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
@@ -111,7 +111,12 @@ exports[`Results > renders a match to the previous snapshot 1`] = `
           Share results
         </h3>
         <p>
-          Copy or email these results. Your answers will be visible. Only share with those you trust.
+          Copy or email these results. Your answers will be visible. 
+          <strong>
+            Only share with those you trust
+            <!--&lt;strong-->
+            .
+          </strong>
         </p>
         <ul
           class="bf-result-view-share-results-button-group"
diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx b/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
index d8a8cb37..e99059f2 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/index.jsx
@@ -110,7 +110,7 @@ const Results = ({
         >
           {shareResults?.heading}
         </Heading>
-        <p>{shareResults?.description}</p>
+        <p dangerouslySetInnerHTML={createMarkup(shareResults?.description)} />
         <ul className="bf-result-view-share-results-button-group">
           <li>
             {' '}
diff --git a/benefit-finder/src/shared/locales/en/en.json b/benefit-finder/src/shared/locales/en/en.json
index 584b6f9d..35c49e7a 100644
--- a/benefit-finder/src/shared/locales/en/en.json
+++ b/benefit-finder/src/shared/locales/en/en.json
@@ -164,7 +164,7 @@
       "shareLinkContent": "Copied",
       "emailTrigger": "Email results",
       "emailSubject": "Results from USAGov’s benefit finder",
-      "description": "Copy or email these results. Your answers will be visible. Only share with those you trust."
+      "description": "Copy or email these results. Your answers will be visible. <strong>Only share with those you trust</<strong>."
     }
   },
   "shareResults": {
diff --git a/benefit-finder/src/shared/locales/es/es.json b/benefit-finder/src/shared/locales/es/es.json
index 435f780b..8ad3f254 100644
--- a/benefit-finder/src/shared/locales/es/es.json
+++ b/benefit-finder/src/shared/locales/es/es.json
@@ -165,7 +165,7 @@
       "emailTrigger": "Envie sus resultados por correo electrónico",
       "shareLinkContent": "Copiado",
       "emailSubject": "Resultados del buscador de beneficios de USAGov",
-      "description": "Copie o envíe sus resultados. Sus respuestas serán visibles. Solo comparta con quienes usted confíe."
+      "description": "Copie o envíe sus resultados. Sus respuestas serán visibles. <strong>Solo comparta con quienes usted confíe</strong>."
     }
   },
   "shareResults": {

From a580dc5f9b07c2dd68e4546adbf2fe1de6820cb6 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:09:02 -0500
Subject: [PATCH 4/8] PXBF-2016-design-qa-tweaks: convert to manual control of
 interactive element for triggers

---
 .../__tests__/__snapshots__/index.spec.jsx.snap      | 12 ++++++++----
 .../__tests__/__snapshots__/index.spec.jsx.snap      |  6 ++++--
 .../__tests__/__snapshots__/index.spec.jsx.snap      |  3 ++-
 .../src/shared/components/EmailTrigger/index.jsx     |  4 +++-
 .../__tests__/__snapshots__/index.spec.jsx.snap      |  3 ++-
 .../src/shared/components/ShareTrigger/index.jsx     |  4 +++-
 6 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
index 9595482c..e69461b1 100644
--- a/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/Routes/ResultsView/__tests__/__snapshots__/index.spec.jsx.snap
@@ -6318,7 +6318,8 @@ exports[`loads view 1`] = `
                 <a
                   class="bf-share-trigger bf-usa-link usa-link"
                   data-testid="bf-share-trigger"
-                  href=""
+                  role="link"
+                  tabindex="0"
                 >
                   Share link
                 </a>
@@ -6327,7 +6328,8 @@ exports[`loads view 1`] = `
                  
                 <a
                   class="bf-email-trigger bf-usa-link usa-link"
-                  href=""
+                  role="link"
+                  tabindex="0"
                 >
                   Email results
                 </a>
@@ -12662,7 +12664,8 @@ exports[`scenario 1 loads in view with the correct amount of likely eligible ite
                 <a
                   class="bf-share-trigger bf-usa-link usa-link"
                   data-testid="bf-share-trigger"
-                  href=""
+                  role="link"
+                  tabindex="0"
                 >
                   Share link
                 </a>
@@ -12671,7 +12674,8 @@ exports[`scenario 1 loads in view with the correct amount of likely eligible ite
                  
                 <a
                   class="bf-email-trigger bf-usa-link usa-link"
-                  href=""
+                  role="link"
+                  tabindex="0"
                 >
                   Email results
                 </a>
diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
index 9b0726c2..80b83d4e 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/__tests__/__snapshots__/index.spec.jsx.snap
@@ -126,7 +126,8 @@ exports[`Results > renders a match to the previous snapshot 1`] = `
             <a
               class="bf-share-trigger bf-usa-link usa-link"
               data-testid="bf-share-trigger"
-              href=""
+              role="link"
+              tabindex="0"
             >
               Share link
             </a>
@@ -135,7 +136,8 @@ exports[`Results > renders a match to the previous snapshot 1`] = `
              
             <a
               class="bf-email-trigger bf-usa-link usa-link"
-              href=""
+              role="link"
+              tabindex="0"
             >
               Email results
             </a>
diff --git a/benefit-finder/src/shared/components/EmailTrigger/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/EmailTrigger/__tests__/__snapshots__/index.spec.jsx.snap
index db533466..492ed90b 100644
--- a/benefit-finder/src/shared/components/EmailTrigger/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/EmailTrigger/__tests__/__snapshots__/index.spec.jsx.snap
@@ -4,7 +4,8 @@ exports[`Email > renders a match to the previous snapshot 1`] = `
 <DocumentFragment>
   <a
     class="bf-email-trigger bf-usa-link usa-link"
-    href=""
+    role="link"
+    tabindex="0"
   >
     Email
   </a>
diff --git a/benefit-finder/src/shared/components/EmailTrigger/index.jsx b/benefit-finder/src/shared/components/EmailTrigger/index.jsx
index c30941d7..54058f09 100644
--- a/benefit-finder/src/shared/components/EmailTrigger/index.jsx
+++ b/benefit-finder/src/shared/components/EmailTrigger/index.jsx
@@ -38,9 +38,11 @@ const EmailTrigger = ({ ui, data }) => {
 
   return (
     <a
-      href=""
+      tabIndex={0}
+      role="link"
       className="bf-email-trigger bf-usa-link usa-link"
       onClick={e => handleClick(e)}
+      onKeyDown={e => handleClick(e)}
     >
       {ui?.emailTrigger || 'Email'}
     </a>
diff --git a/benefit-finder/src/shared/components/ShareTrigger/__tests__/__snapshots__/index.spec.jsx.snap b/benefit-finder/src/shared/components/ShareTrigger/__tests__/__snapshots__/index.spec.jsx.snap
index d1534ea1..8f0abec1 100644
--- a/benefit-finder/src/shared/components/ShareTrigger/__tests__/__snapshots__/index.spec.jsx.snap
+++ b/benefit-finder/src/shared/components/ShareTrigger/__tests__/__snapshots__/index.spec.jsx.snap
@@ -5,7 +5,8 @@ exports[`Share > renders a match to the previous snapshot 1`] = `
   <a
     class="bf-share-trigger bf-usa-link usa-link"
     data-testid="bf-share-trigger"
-    href=""
+    role="link"
+    tabindex="0"
   >
     Share
   </a>
diff --git a/benefit-finder/src/shared/components/ShareTrigger/index.jsx b/benefit-finder/src/shared/components/ShareTrigger/index.jsx
index 08aa2f55..9f81d9cc 100644
--- a/benefit-finder/src/shared/components/ShareTrigger/index.jsx
+++ b/benefit-finder/src/shared/components/ShareTrigger/index.jsx
@@ -40,9 +40,11 @@ const ShareTrigger = ({ ui, data }) => {
 
   return (
     <a
-      href=""
+      tabIndex={0}
+      role="link"
       className="bf-share-trigger bf-usa-link usa-link"
       onClick={e => handleClick(e)}
+      onKeyDown={e => handleClick(e)}
       data-testid="bf-share-trigger"
     >
       {ui?.shareTrigger || 'Share'}

From 19f6d962567b12508c416c64d0c216c3d41e96a4 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:10:59 -0500
Subject: [PATCH 5/8] PXBF-2016-design-qa-tweaks: add more specificity to
 mobile adjustment on unordered lists

---
 benefit-finder/src/shared/components/Alert/_index.scss | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/benefit-finder/src/shared/components/Alert/_index.scss b/benefit-finder/src/shared/components/Alert/_index.scss
index c32ebb48..19d5787f 100644
--- a/benefit-finder/src/shared/components/Alert/_index.scss
+++ b/benefit-finder/src/shared/components/Alert/_index.scss
@@ -3,7 +3,6 @@
 @use '@styles/breakpoints' as *;
 @use '@styles/functions' as *;
 
-
 h2.bf-usa-alert__heading {
   @include h5($font-family: 'sans');
 
@@ -31,7 +30,7 @@ h2.bf-usa-alert__heading {
     padding-left: rem(32px);
   }
 
-  ul {
+  .bf-usa-alert ul {
     padding-left: 0;
   }
 

From e846838716a6e23a9a6e2290a39eb5f4ff235757 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:23:54 -0500
Subject: [PATCH 6/8] PXBF-2016-design-qa-tweaks: update margin for heading and
 descriptions on results view mobile

---
 .../ResultsView/components/Results/_index.scss | 18 ++++++++++++++++++
 .../blocks/EligibleBenefitsHeading/_index.scss | 14 ++++++++++++++
 .../src/shared/components/Summary/_index.scss  |  7 ++++++-
 3 files changed, 38 insertions(+), 1 deletion(-)

diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
index 08c69a62..f0dc8bf9 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
@@ -20,6 +20,20 @@
   .bf-result-view-relevant-benefits-heading,
   .bf-result-view-share-results-heading {
     @include h3;
+
+    margin-left: rem(15px);
+
+    @media (width > $desktop) {
+      margin-left: 0;
+    }
+  }
+
+  p {
+    margin-left: rem(15px);
+
+    @media (width > $desktop) {
+      margin-left: 0;
+    }
   }
 
   .bf-expand-all {
@@ -36,6 +50,10 @@
     display: flex;
     justify-content: center;
   }
+
+  @media (width > $desktop) {
+    margin-left: 0;
+  }
 }
 
 .bf-result-view-share-results-trigger-group {
diff --git a/benefit-finder/src/Routes/ResultsView/components/blocks/EligibleBenefitsHeading/_index.scss b/benefit-finder/src/Routes/ResultsView/components/blocks/EligibleBenefitsHeading/_index.scss
index e2b813fd..905e6c85 100644
--- a/benefit-finder/src/Routes/ResultsView/components/blocks/EligibleBenefitsHeading/_index.scss
+++ b/benefit-finder/src/Routes/ResultsView/components/blocks/EligibleBenefitsHeading/_index.scss
@@ -1,9 +1,23 @@
 @use '@styles/mixins' as *;
+@use '@styles/functions' as *;
+@use '@styles/breakpoints' as *;
 
 .bf-eligible-view-heading {
   @include h2;
+
+  margin-left: rem(15px);
+
+  @media (width > $desktop) {
+    margin-left: 0;
+  }
 }
 
 .bf-eligible-view-description {
   @include h6;
+
+  margin-left: rem(15px);
+
+  @media (width > $desktop) {
+    margin-left: 0;
+  }
 }
diff --git a/benefit-finder/src/shared/components/Summary/_index.scss b/benefit-finder/src/shared/components/Summary/_index.scss
index 3244bfb7..03242c72 100644
--- a/benefit-finder/src/shared/components/Summary/_index.scss
+++ b/benefit-finder/src/shared/components/Summary/_index.scss
@@ -1,13 +1,18 @@
 @use '@styles/mixins' as *;
 @use '@styles/colors' as color;
 @use '@styles/functions' as *;
+@use '@styles/breakpoints' as *;
 
 .bf-usa-summary-box {
   @include h5($font-family: 'sans');
 
-  margin-bottom: rem(16px);
+  margin: rem(16px) rem(15px);
   border-color: color.$teal;
 
+  @media (width > $desktop) {
+    margin: rem(16px) 0;
+  }
+
   .bf-usa-list {
     li {
       @include list-items;

From 11697f1bc6a80ea5fcea6f05d0f3ff57aca03648 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:33:35 -0500
Subject: [PATCH 7/8] PXBF-2016-design-qa-tweaks: adjust to first of type p in
 descendent

---
 .../src/Routes/ResultsView/components/Results/_index.scss       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
index f0dc8bf9..08d27771 100644
--- a/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
+++ b/benefit-finder/src/Routes/ResultsView/components/Results/_index.scss
@@ -28,7 +28,7 @@
     }
   }
 
-  p {
+  > p {
     margin-left: rem(15px);
 
     @media (width > $desktop) {

From 0d27aa374a973420429bfbb7d2703bb8d221d8c6 Mon Sep 17 00:00:00 2001
From: scottqueen-bixal <scott.queen@bixal.com>
Date: Mon, 23 Dec 2024 16:44:37 -0500
Subject: [PATCH 8/8] PXBF-2016-design-qa-tweaks: set default width to desktop

---
 benefit-finder/cypress.config.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/benefit-finder/cypress.config.js b/benefit-finder/cypress.config.js
index 74db8b19..672f2c41 100644
--- a/benefit-finder/cypress.config.js
+++ b/benefit-finder/cypress.config.js
@@ -6,6 +6,7 @@ module.exports = defineConfig({
     runMode: 2,
     openMode: 0,
   },
+  viewportWidth: 1050,
   e2e: {
     baseUrl: 'http://localhost:6006',
     excludeSpecPattern: 'cypress/e2e/usagov-public-site/*.cy.js',