From b317bd7a0be688ebac18d3680d5e2c906229b737 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Thu, 22 Nov 2018 12:16:32 +0000
Subject: [PATCH] Put download PDF in footer

This makes its positioning consistent with the previous page in the
one-off sending journey.

It gives us more space to put information about the status of the letter
above the preview of the letter.
---
 .../stick-at-top-when-scrolling.scss          |  1 +
 .../views/notifications/notification.html     | 22 ++++++++++---------
 2 files changed, 13 insertions(+), 10 deletions(-)

diff --git a/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss b/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss
index 283c1aeec..34bb42526 100644
--- a/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss
+++ b/app/assets/stylesheets/components/stick-at-top-when-scrolling.scss
@@ -48,6 +48,7 @@
 
   .page-footer {
     margin-bottom: 0;
+    min-height: 50px;
   }
 
   .notification-status {
diff --git a/app/templates/views/notifications/notification.html b/app/templates/views/notifications/notification.html
index 7fc689cac..d0e8c9613 100644
--- a/app/templates/views/notifications/notification.html
+++ b/app/templates/views/notifications/notification.html
@@ -51,22 +51,24 @@
         <p>
           Estimated delivery date: {{ estimated_letter_delivery_date|string|format_date_short }}
         </p>
-        <p class="bottom-gutter">
-          <a href="{{ url_for('main.view_letter_notification_as_preview', service_id=current_service.id, notification_id=notification_id, filetype='pdf') }}" download>Download as a PDF</a>
-        </p>
       {% endif %}
     {% endif %}
 
     {{ template|string }}
 
-    {% if template.template_type != 'letter' %}
-
-      {% if template.template_type == 'email' %}<div class="js-stick-at-bottom-when-scrolling">{% endif %}
-
+    {% if template.template_type == 'letter' %}
+      <div class="js-stick-at-bottom-when-scrolling">
+        <div class="page-footer">
+          <div>&nbsp;</div>
+          <a class="page-footer-right-aligned-link" href="{{ url_for('main.view_letter_notification_as_preview', service_id=current_service.id, notification_id=notification_id, filetype='pdf') }}" download>Download as a PDF</a>
+        </div>
+      </div>
+    {% elif template.template_type == 'email' %}
+      <div class="js-stick-at-bottom-when-scrolling">
+        {{ ajax_block(partials, updates_url, 'status', finished=finished) }}
+      </div>
+    {% elif template.template_type == 'sms' %}
       {{ ajax_block(partials, updates_url, 'status', finished=finished) }}
-
-      {% if template.template_type == 'email' %}</div>{% endif %}
-
     {% endif %}
 
     {% if current_user.has_permissions('send_messages') and current_user.has_permissions('view_activity') and template.template_type == 'sms' and can_receive_inbound %}