darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-14 02:48:45 -04:00
Code Issues Packages Projects Releases Wiki Activity

Format ‘pending’ messages the same as ‘sending’

Browse Source 
For reasons I don’t understand some of our text messages get marked as `pending`. Since we don’t account for this in the front end they don’t get displayed quite right (the vertical position of the text is too high). This commit adds the pending `state` to the `if` statement so the text is properly aligned.
This commit is contained in:
Chris Hill-Scott
2018-10-10 09:50:14 +01:00
parent 6f38a5030b
commit 0d0d95a75a
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/templates/components/table.html
View File

@@ -143,6 +143,9 @@
{% macro notification_status_field(notification) %}
{% set displayed_on_single_line = notification.status in ['created', 'pending', 'sending', 'delivered'] %}
{% if not notification %}
{% call field(align='right') %}{% endcall %}
{% else %}
@@ -150,7 +153,7 @@
status=notification.status|format_notification_status_as_field_status(notification.notification_type),
align='right'
) %}
{% if notification.status in ['created', 'sending', 'delivered'] %}<span class="align-with-message-body">{% endif %}
{% if displayed_on_single_line %}<span class="align-with-message-body">{% endif %}
{% if notification.status|format_notification_status_as_url %}
<a href="{{ notification.status|format_notification_status_as_url }}">
{% endif %}
@@ -172,7 +175,7 @@
) }}
{% endif %}
</span>
{% if notification.status in ['created', 'sending', 'delivered'] %}</span>{% endif %}
{% if displayed_on_single_line %}</span>{% endif %}
{% endcall %}
{% endif %}
{% endmacro %}