Fixing styling issues on the send message flow

app/templates/components/folder-path.html

@@ -6,7 +6,7 @@
   link_current_item=False,
   root_element='h1'
 ) %}
-  <{{ root_element }} class="font-body-lg folder-heading"{% if root_element == 'h1' %} id="page-header"{% endif %}>
+  <{{ root_element }} class="font-body-lg folder-heading margin-bottom-0"{% if root_element == 'h1' %} id="page-header"{% endif %}>
     {% for folder in folders %}
       {% if loop.last and not link_current_item %}
         {% if folder.template_type or not folder.id %}

app/templates/partials/count.html

@@ -3,7 +3,7 @@
 
 <div class="ajax-block-container">
   {% if notifications_deleted %}
-    <div class="grid-row bottom-gutter-1-2">
+    <div class="grid-row ">
       {% for label, query_param, url, count in counts %}
         {% if query_param == 'pending' %}
             <div class="grid-col-3">{{ big_number(count, query_param, smaller=True) }}</div>
@@ -14,7 +14,7 @@
       {% endfor %}
     </div>
   {% else %}
-    <div class="bottom-gutter">
+    <div class="tabs">
       {{ pill(counts, request.args.get('status', '')) }}
     </div>
   {% endif %}

app/templates/views/api/index.html

@@ -12,7 +12,7 @@
     API integration
   </h1>
 
-  <nav class="grid-row bottom-gutter-1-2">
+  <nav class="grid-row">
     <div class="grid-col-4">
       <a class="usa-link pill-separate-item"  href="{{ url_for('.api_keys', service_id=current_service.id) }}">API keys</a>
     </div>

app/templates/views/check/column-errors.html

@@ -15,7 +15,7 @@
 
 {% block maincolumn_content %}
 
-  <div class="bottom-gutter-1-2">
+  <div class="">
     {% call banner_wrapper(type='dangerous') %}
 
       {% if recipients.too_many_rows %}

app/templates/views/check/ok.html

@@ -21,11 +21,6 @@
 
   {{ page_header('Preview of {}'.format(template.name)) }}
 
-  {{ usaSkipLink({
-    "text": "Skip to file contents",
-    "href": "#" + file_contents_header_id
-  }) }}
-
   {{ template|string }}
   <div class="bottom-gutter-3-2">
     <form method="post" enctype="multipart/form-data" action="{{url_for('main.start_job', service_id=current_service.id, upload_id=upload_id)}}" class='page-footer'>

app/templates/views/check/row-errors.html

@@ -4,6 +4,7 @@
 {% from "components/table.html" import mapping_table, row, field, text_field, index_field, hidden_field_heading %}
 {% from "components/file-upload.html" import file_upload %}
 {% from "components/components/back-link/macro.njk" import usaBackLink %}
+{% from "components/components/alert/macro.njk" import usaAlert %}
 
 {% block service_page_title %}
   Error
@@ -15,27 +16,31 @@
 
 {% block maincolumn_content %}
 
-  <div class="bottom-gutter-1-2">
+  <div class="">
     {% call banner_wrapper(type='dangerous') %}
       {% if row_errors|length == 1 %}
-        <h1 class='banner-title' data-module="track-error" data-error-type="Bad rows" data-error-label="{{ upload_id }}">
-          There’s a problem with {{ original_file_name }}
-        </h1>
-        <p>
-          You need to {{ row_errors[0] }}.
-        </p>
+        <div class="usa-alert usa-alert--error" role="alert">
+          <div class="usa-alert__body">
+            <h1 class="usa-alert__heading">There’s a problem with {{ original_file_name }}</h1>
+            <p class="usa-alert__text">
+            You need to {{ row_errors[0] }}.
+            </p>
+          </div>
+        </div>
       {% else %}
-        <h1 class='banner-title' data-module="track-error" data-error-type="Bad rows" data-error-label="{{ upload_id }}">
-          There are some problems with {{ original_file_name }}
-        </h1>
-        <p>
-          You need to:
-        </p>
-        <ul class="list-bullet">
-          {% for error in row_errors %}
-            <li>{{ error }}</li>
-          {% endfor %}
-        </ul>
+        <div class="usa-alert usa-alert--error" role="alert">
+          <div class="usa-alert__body">
+            <h1 class="usa-alert__heading">There’s a problem with {{ original_file_name }}</h1>
+            <p class="usa-alert__text">
+              You need to:
+              <ul class="list-bullet">
+                {% for error in row_errors %}
+                <li>{{ error }}</li>
+                {% endfor %}
+              </ul>
+            </p>
+          </div>
+        </div>
       {% endif %}
     {% endcall %}
   </div>
@@ -49,7 +54,6 @@
         button_text='Upload your file again'
       ) }}
     </div>
-    <a href="#content" class="usa-link back-to-top-link">Back to top</a>
   </div>
 
   <div class="fullscreen-content" data-module="fullscreen-table">

app/templates/views/dashboard/dashboard.html

@@ -19,7 +19,7 @@
 
     {{ ajax_block(partials, updates_url, 'upcoming') }}
 
-    <h2 class="font-body-lg margin-top-0">
+    <h2 class="font-body-xl margin-top-0 margin-bottom-1">
       In the last seven days
     </h2>
 

app/templates/views/send-test.html

@@ -28,9 +28,9 @@
         {{ form.placeholder_value(param_extensions={"classes": ""}) }}
       </div>
       {% if skip_link or link_to_upload %}
-        <div class="grid-col-12">
+        <div class="grid-col-12 margin-top-1">
           {% if link_to_upload %}
-            <a class="usa-link" href="{{ url_for('.send_messages', service_id=current_service.id, template_id=template.id) }}">Upload a list of {{ 999|recipient_count_label(template.template_type) }}</a>
+            <a class="usa-link margin-right-2" href="{{ url_for('.send_messages', service_id=current_service.id, template_id=template.id) }}">Upload a list of {{ 999|recipient_count_label(template.template_type) }}</a>
           {% endif %}
           {% if skip_link %}
             <a href="{{ skip_link[1] }}" class="usa-link">{{ skip_link[0] }}</a>

app/templates/views/send.html

@@ -53,7 +53,7 @@
     <a class="usa-link" href="{{ url_for('.get_example_csv', service_id=current_service.id, template_id=template.id) }}" download>Download this example (<abbr title="Comma separated values">CSV</abbr>)</a>
   </p>
 
-  <h2 class="font-body-lg">Your file will populate this template ({{ template.name }})</h2>
+  <h2 class="font-body-lg margin-bottom-1">Your file will populate this template:<br><span class="font-body-lg">({{ template.name }})</span></h2>
   {{ template|string }}
 
 {% endblock %}

app/templates/views/service-settings.html

@@ -373,7 +373,7 @@
     {% endif %}
     {% if (not current_service.active) and current_user.platform_admin %}
       <p>
-        <div class="hint bottom-gutter-1-2">
+        <div class="hint ">
           Service suspended
         </div>
 

app/templates/views/templates/choose-reply.html

@@ -10,7 +10,7 @@
 
 {% block maincolumn_content %}
 
-<div class="bottom-gutter-1-2">
+<div class="">
   <h1 class="font-body-2xl margin-bottom-3">Choose a template</h1>
   {{ folder_path(template_folder_path, current_service.id, template_type, current_user, root_element='h2') }}
 </div>

app/templates/views/templates/copy.html

@@ -10,7 +10,7 @@
 
 {% block maincolumn_content %}
 
-    <div class="bottom-gutter-1-2">
+    <div class="">
       <h1 class="font-body-2xl margin-bottom-3">{{ page_title }}</h1>
       {{ copy_folder_path(template_folder_path, current_service.id, from_service, current_user) }}
     </div>

app/templates/views/templates/manage-template-folder.html

@@ -9,7 +9,7 @@
 
 {% block maincolumn_content %}
 
-  <div class="grid-row bottom-gutter-1-2">
+  <div class="grid-row ">
     <div class="grid-col-12">
       {{ folder_path(
         folders=template_folder_path,

app/templates/views/templates/template.html

@@ -32,7 +32,7 @@
   {% else %}
     <div class="grid-row">
       <div class="grid-col-12">
-        <h1 class="font-body-lg folder-heading">
+        <h1 class="font-body-xl folder-heading margin-top-0">
           Review your message
         </h1>
         {{ folder_path(
@@ -70,7 +70,7 @@
     {% endif %}
   </div>
 
-  <!--<div class="bottom-gutter-1-2">
+  <!--<div class="">
     {{ copy_to_clipboard(template.id, name="Template ID", thing='template ID') }}
   </div>-->