From 378a6e4452038dc26dad84f7702bcd81e060a2b1 Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Thu, 19 Sep 2024 19:27:53 -0400
Subject: [PATCH 1/6] fix for font saturation

---
 .../templates/includes/summary_item.html      | 33 ++++++++++---------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index f69bbaf96..93aada291 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -7,19 +7,15 @@
     {% if heading_level %}
       <{{ heading_level }}
     {% else %} 
-      <h3
+      <h4
     {% endif %} 
-      class="summary-item__title
-             font-sans-md
-             text-primary-dark text-semibold
-             margin-top-0 margin-bottom-05
-             padding-right-1"
-      >
+      class="font-sans-md
+             text-primary-dark text-semibold">
         {{ title }}
     {% if heading_level %}
       </{{ heading_level }}>
     {% else %} 
-      </h3>
+      </h4>
     {% endif %}
       {% if sub_header_text %}
         <h4 class="register-form-review-header">{{ sub_header_text }}</h4>
@@ -32,16 +28,23 @@ <h4 class="register-form-review-header">{{ sub_header_text }}</h4>
             {% include "includes/contact.html" with contact=value|first %}
           {% else %}
             {% if value %}
-              <dl class="usa-list usa-list--unstyled margin-top-0">
+              <ul class="usa-list usa-list--unstyled margin-top-0">
                 {% for item in value %}
-                  <dt>
-                    Contact {{forloop.counter}}
-                  </dt>
-                  <dd>
+                  <li>
+                  
+                 <h4 class="summary-item__title
+             font-sans-md
+             text-primary-dark text-semibold
+             margin-bottom-05
+             padding-right-1">
+                      Contact {{forloop.counter}}
+                  </h4>
+                  </li>
+                  <li>
                     {% include "includes/contact.html" with contact=item %}
-                  </dd>
+                  </li>
                 {% endfor %}
-                </dl>
+                </ul>
             {% elif custom_text_for_value_none %}
               <p>
                 {{ custom_text_for_value_none }}

From a0e745824fd97ce17f3ae1da1706d71471abb83a Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Fri, 20 Sep 2024 14:55:11 -0400
Subject: [PATCH 2/6] reverted a change

---
 src/registrar/templates/includes/summary_item.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index 93aada291..11e1d971b 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -7,7 +7,7 @@
     {% if heading_level %}
       <{{ heading_level }}
     {% else %} 
-      <h4
+      <h3
     {% endif %} 
       class="font-sans-md
              text-primary-dark text-semibold">
@@ -15,7 +15,7 @@
     {% if heading_level %}
       </{{ heading_level }}>
     {% else %} 
-      </h4>
+      </h3>
     {% endif %}
       {% if sub_header_text %}
         <h4 class="register-form-review-header">{{ sub_header_text }}</h4>

From 36e50cb61af2a999a0fd64f0b9d63f9ea35f4b3c Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Fri, 20 Sep 2024 16:05:05 -0400
Subject: [PATCH 3/6] reverted to dt/dl list

---
 src/registrar/templates/includes/summary_item.html | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index 11e1d971b..d22582da7 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -28,9 +28,9 @@ <h4 class="register-form-review-header">{{ sub_header_text }}</h4>
             {% include "includes/contact.html" with contact=value|first %}
           {% else %}
             {% if value %}
-              <ul class="usa-list usa-list--unstyled margin-top-0">
+              <dl class="usa-list usa-list--unstyled margin-top-0">
                 {% for item in value %}
-                  <li>
+                  <dt>
                   
                  <h4 class="summary-item__title
              font-sans-md
@@ -39,12 +39,12 @@ <h4 class="register-form-review-header">{{ sub_header_text }}</h4>
              padding-right-1">
                       Contact {{forloop.counter}}
                   </h4>
-                  </li>
-                  <li>
+                </dt>
+                  <dd>
                     {% include "includes/contact.html" with contact=item %}
-                  </li>
+                  </dd>
                 {% endfor %}
-                </ul>
+                </dl>
             {% elif custom_text_for_value_none %}
               <p>
                 {{ custom_text_for_value_none }}

From f974c64b328a4df29579582fc29448dd80d0a795 Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Fri, 20 Sep 2024 16:06:18 -0400
Subject: [PATCH 4/6] added back styling

---
 src/registrar/templates/includes/summary_item.html | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index d22582da7..a481b7a80 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -9,8 +9,11 @@
     {% else %} 
       <h3
     {% endif %} 
-      class="font-sans-md
-             text-primary-dark text-semibold">
+      class="summary-item__title
+             font-sans-md
+             text-primary-dark text-semibold
+             margin-top-0 margin-bottom-05
+             padding-right-1">
         {{ title }}
     {% if heading_level %}
       </{{ heading_level }}>

From e12c406e2ee16f9da1303d34b0a95b8f130326e1 Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Fri, 20 Sep 2024 16:07:04 -0400
Subject: [PATCH 5/6] fixed spacing

---
 src/registrar/templates/includes/summary_item.html | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index a481b7a80..29d366f6c 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -13,7 +13,8 @@
              font-sans-md
              text-primary-dark text-semibold
              margin-top-0 margin-bottom-05
-             padding-right-1">
+             padding-right-1"
+             >
         {{ title }}
     {% if heading_level %}
       </{{ heading_level }}>

From 2e028a0f44ac2e60c0cfe6ac21c51927580b3461 Mon Sep 17 00:00:00 2001
From: asaki222 <asaki444@gmail.com>
Date: Fri, 20 Sep 2024 16:07:40 -0400
Subject: [PATCH 6/6] fixed spacing again O.o

---
 src/registrar/templates/includes/summary_item.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/registrar/templates/includes/summary_item.html b/src/registrar/templates/includes/summary_item.html
index 29d366f6c..d4c68395f 100644
--- a/src/registrar/templates/includes/summary_item.html
+++ b/src/registrar/templates/includes/summary_item.html
@@ -14,7 +14,7 @@
              text-primary-dark text-semibold
              margin-top-0 margin-bottom-05
              padding-right-1"
-             >
+      >
         {{ title }}
     {% if heading_level %}
       </{{ heading_level }}>