diff --git a/app/assets/stylesheets/components/page-footer.scss b/app/assets/stylesheets/components/page-footer.scss index 31e7e5ab0..04b6a31b1 100644 --- a/app/assets/stylesheets/components/page-footer.scss +++ b/app/assets/stylesheets/components/page-footer.scss @@ -39,6 +39,10 @@ } + &-delete-link-without-button { + padding-left: 0; + } + &-secondary-link { display: block; margin-top: $gutter; diff --git a/app/main/views/templates.py b/app/main/views/templates.py index e034f5647..7a9e94beb 100644 --- a/app/main/views/templates.py +++ b/app/main/views/templates.py @@ -311,12 +311,8 @@ def delete_service_template(service_id, template_id): return redirect(url_for( '.choose_template', service_id=service_id, - template_type=template['template_type'] )) - template['template_content'] = template['content'] - form = form_objects[template['template_type']](**template) - try: last_used_notification = template_statistics_client.get_template_statistics_for_template( service_id, template['id'] @@ -334,11 +330,17 @@ def delete_service_template(service_id, template_id): raise e flash('{}. Are you sure you want to delete it?'.format(message), 'delete') + return render_template( - 'views/edit-{}-template.html'.format(template['template_type']), - h1='Edit template', - form=form, - template_id=template_id) + 'views/templates/template.html', + template=get_template( + template, + current_service, + expand_emails=True, + letter_preview_url=url_for('.view_template', service_id=service_id, template_id=template['id']), + show_recipient=True, + ) + ) @main.route('/services//templates//versions') diff --git a/app/templates/components/page-footer.html b/app/templates/components/page-footer.html index bdd5be23f..5dcc69e4f 100644 --- a/app/templates/components/page-footer.html +++ b/app/templates/components/page-footer.html @@ -17,7 +17,7 @@ {{ back_link_text }} {% endif %} {% if delete_link %} - + {{ delete_link_text }} {% endif %} diff --git a/app/templates/flash_messages.html b/app/templates/flash_messages.html index ebb2463d4..0107749a1 100644 --- a/app/templates/flash_messages.html +++ b/app/templates/flash_messages.html @@ -2,12 +2,14 @@ {% with messages = get_flashed_messages(with_categories=true) %} {% if messages %} {% for category, message in messages %} - {{ banner( - message, - 'default' if ((category == 'default') or (category == 'default_with_tick')) else 'dangerous', - delete_button="Yes, {}".format(category) if category in ['delete', 'suspend', 'resume', 'remove'] else None, - with_tick=True if category == 'default_with_tick' else False - )}} +
+ {{ banner( + message, + 'default' if ((category == 'default') or (category == 'default_with_tick')) else 'dangerous', + delete_button="Yes, {}".format(category) if category in ['delete', 'suspend', 'resume', 'remove'] else None, + with_tick=True if category == 'default_with_tick' else False + )}} +
{% endfor %} {% endif %} {% endwith %} diff --git a/app/templates/views/edit-email-template.html b/app/templates/views/edit-email-template.html index 0319820a1..a309b1e00 100644 --- a/app/templates/views/edit-email-template.html +++ b/app/templates/views/edit-email-template.html @@ -23,9 +23,7 @@ {{ radios(form.process_type) }} {% endif %} {{ page_footer( - 'Save', - delete_link=url_for('.delete_service_template', service_id=current_service.id, template_id=template_id) if template_id or None, - delete_link_text='Delete this template' + 'Save' ) }}