From 23afe6fe3614f128f3b8c0280ec53d7713809605 Mon Sep 17 00:00:00 2001
From: Tom Byers <tombaromba@gmail.com>
Date: Wed, 26 Aug 2020 15:48:15 +0100
Subject: [PATCH] Fix letter contact details page

---
 .../letter-contact-details.html               | 51 +++++++++++--------
 1 file changed, 31 insertions(+), 20 deletions(-)

diff --git a/app/templates/views/service-settings/letter-contact-details.html b/app/templates/views/service-settings/letter-contact-details.html
index 0cf921156..a16ea0e1e 100644
--- a/app/templates/views/service-settings/letter-contact-details.html
+++ b/app/templates/views/service-settings/letter-contact-details.html
@@ -18,30 +18,41 @@
   </div>
   <div class="user-list">
     <div class="user-list-item">
-      <span class="hint">
-        Blank
-        {% if current_service.default_letter_contact_block %}
-          {% if current_user.has_permissions('manage_service') %}
-            <a class="govuk-link govuk-link--no-visited-state user-list-edit-link" href="{{ url_for('.service_make_blank_default_letter_contact', service_id =current_service.id) }}">Make default</a>
+      <div class="govuk-grid-row">
+        <div class="govuk-grid-column-three-quarters">
+          <p class="user-list-item-blank govuk-body hint">
+            Blank
+            {% if not current_service.default_letter_contact_block %}
+              (default)
+            {% endif %}
+          </p>
+        </div>
+        <div class="govuk-grid-column-one-quarter">
+          {% if current_service.default_letter_contact_block and current_user.has_permissions('manage_service') %}
+              <a class="govuk-link govuk-link--no-visited-state user-list-edit-link" href="{{ url_for('.service_make_blank_default_letter_contact', service_id =current_service.id) }}">Make default</a>
           {% endif %}
-        {% else %}
-          (default)
-        {% endif %}
-      </span>
+        </div>
+      </div>
     </div>
     {% for item in letter_contact_details %}
       <div class="user-list-item">
-        <p class="govuk-body">
-          {{ item.contact_block | nl2br }}
-        </p>
-        <p class="hint">
-          {%- if item.is_default -%}
-            (default)
-          {% endif %}
-        </p>
-        {% if current_user.has_permissions('manage_service') %}
-        <a class="govuk-link govuk-link--no-visited-state user-list-edit-link" href="{{ url_for('.service_edit_letter_contact', service_id =current_service.id, letter_contact_id = item.id) }}">Change</a>
-        {% endif %}
+        <div class="govuk-grid-row">
+          <div class="govuk-grid-column-three-quarters">
+            <p class="govuk-body">
+              {{ item.contact_block | nl2br }}
+            </p>
+            <p class="govuk-body hint">
+              {%- if item.is_default -%}
+                (default)
+              {% endif %}
+            </p>
+          </div>
+          <div class="govuk-grid-column-one-quarter">
+            {% if current_user.has_permissions('manage_service') %}
+            <a class="govuk-link govuk-link--no-visited-state user-list-edit-link" href="{{ url_for('.service_edit_letter_contact', service_id =current_service.id, letter_contact_id = item.id) }}">Change</a>
+            {% endif %}
+          </div>
+        </div>
         {% if letter_contact_details|length  > 1 %}
           {% set first_line_of_contact_block = item.contact_block|normalise_lines|first %}
           {{ api_key(item.id, name=first_line_of_contact_block, thing="ID") }}