From 33e1b2679fceb13c890826735d95661c2e073b63 Mon Sep 17 00:00:00 2001
From: Chris Hill-Scott <me@quis.cc>
Date: Mon, 11 Apr 2016 15:18:29 +0100
Subject: [PATCH] =?UTF-8?q?Move=20=E2=80=98edit=20template=E2=80=99=20to?=
 =?UTF-8?q?=20its=20own=20link?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

It’s probably clearer to have it labelled explicitly, rather than have
people infer that they should click the title of a template to edit it.
---
 app/templates/views/choose-template.html | 19 ++++++-------------
 1 file changed, 6 insertions(+), 13 deletions(-)

diff --git a/app/templates/views/choose-template.html b/app/templates/views/choose-template.html
index 15d1ba883..b73124ffb 100644
--- a/app/templates/views/choose-template.html
+++ b/app/templates/views/choose-template.html
@@ -45,22 +45,12 @@
               template.subject,
               template.formatted_as_markup,
               name=template.name,
-              edit_link=(
-                url_for(".edit_service_template", service_id=current_service.id, template_id=template.id)
-                if current_user.has_permissions(permissions=['manage_templates'], admin_override=True) else
-                None
-              )
             ) }}
           {% elif 'sms' == template_type %}
             {{ sms_message(
-                template.formatted_as_markup,
-                name=template.name,
-                edit_link=(
-                  url_for(".edit_service_template", service_id=current_service.id, template_id=template.id)
-                  if current_user.has_permissions(permissions=['manage_templates'], admin_override=True) else
-                  None
-                )
-              ) }}
+              template.formatted_as_markup,
+              name=template.name
+            ) }}
           {% endif %}
         </div>
         <div class="column-one-third">
@@ -72,6 +62,9 @@
             {% if current_user.has_permissions(permissions=['manage_api_keys']) %}
               <a href="{{ url_for(".send_from_api", service_id=current_service.id, template_id=template.id) }}">API integration</a>
             {% endif %}
+            {% if current_user.has_permissions(permissions=['manage_templates'], admin_override=True) %}
+              <a href="{{ url_for(".edit_service_template", service_id=current_service.id, template_id=template.id) }}">Edit template</a>
+            {% endif %}
           </div>
         </div>
       {% endfor %}