diff --git a/app/assets/stylesheets/components/sms-message.scss b/app/assets/stylesheets/components/sms-message.scss index 22d83cf6e..4fa727388 100644 --- a/app/assets/stylesheets/components/sms-message.scss +++ b/app/assets/stylesheets/components/sms-message.scss @@ -42,7 +42,7 @@ &-recipient { @include copy-19; color: $secondary-text-colour; - margin: 0; + margin: -$gutter-half 0 $gutter 0; } &-history { @@ -50,29 +50,20 @@ background: $turquoise; color: $white; padding: $gutter-half; - @include core-16; - margin-bottom: $gutter; - @extend %contain-floats; + @include bold-19; + margin: 0 0 $gutter 0; &-heading { - @include bold-16; - margin-bottom: 10px; - } - &-status { - float: left; - margin: 0 0 10px 0; - padding: 0; - width: 66%; - } + @include bold-19; + margin: 0; + + &-time { + @include inline-block; + margin-left: 10px; + font-weight: normal; + } - &-time { - float: left; - clear: left; - padding: 0; - width: 33%; - min-width: 3.2em; - max-width: 4.8em; } } diff --git a/app/main/views/jobs.py b/app/main/views/jobs.py index 0800c751d..867bb0269 100644 --- a/app/main/views/jobs.py +++ b/app/main/views/jobs.py @@ -75,10 +75,6 @@ def shownotification(notification_id): message=[ message for message in messages if message['id'] == notification_id ][0], - history=[ - {'time': now, 'status': 'Delivered'}, - {'time': now, 'status': 'Received by handset'}, - {'time': now, 'status': 'Accepted by network'}, - {'time': now, 'status': 'Sent'}, - ] + delivered_at=now, + uploaded_at=now ) diff --git a/app/templates/components/sms-message.html b/app/templates/components/sms-message.html index d5f63909d..38d30b877 100644 --- a/app/templates/components/sms-message.html +++ b/app/templates/components/sms-message.html @@ -1,22 +1,20 @@ {% macro sms_message(body, recipient) %} - {% if recipient %} -
- {% endif %} + {% if recipient %} + + {% endif %} {% endmacro %} -{% macro message_history(title, log, heading_level=2) %} - +{% macro message_status(status, time) %} + {% endmacro %} diff --git a/app/templates/views/notification.html b/app/templates/views/notification.html index c22cae954..1dcb034a2 100644 --- a/app/templates/views/notification.html +++ b/app/templates/views/notification.html @@ -1,5 +1,5 @@ {% extends "withnav_template.html" %} -{% from "components/sms-message.html" import sms_message, message_history %} +{% from "components/sms-message.html" import sms_message, message_status %} {% block page_title %} GOV.UK Notify | Notifications activity @@ -17,7 +17,7 @@ GOV.UK Notify | Notifications activity {{ sms_message(message.message, message.phone) }}