From 723f343d05704b63ede7f370fcf22b11719f6069 Mon Sep 17 00:00:00 2001
From: Zarino Zappia <mail@zarino.co.uk>
Date: Thu, 21 Nov 2024 13:36:49 +0000
Subject: [PATCH] Tidier "Edit" action for assignments on volunteer list

---
 crowdsourcer/templates/crowdsourcer/icons/edit.svg       | 1 +
 crowdsourcer/templates/crowdsourcer/volunteers/list.html | 9 ++++++---
 2 files changed, 7 insertions(+), 3 deletions(-)
 create mode 100644 crowdsourcer/templates/crowdsourcer/icons/edit.svg

diff --git a/crowdsourcer/templates/crowdsourcer/icons/edit.svg b/crowdsourcer/templates/crowdsourcer/icons/edit.svg
new file mode 100644
index 0000000..d4766d0
--- /dev/null
+++ b/crowdsourcer/templates/crowdsourcer/icons/edit.svg
@@ -0,0 +1 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="{{ width|default:'1em' }}" height="{{ height|default:'1em' }}" viewBox="0 0 20 20" role="{{ role|default:'presentation' }}" class="{{ classes }}"><path fill="currentColor" d="m16.77 8l1.94-2a1 1 0 0 0 0-1.41l-3.34-3.3a1 1 0 0 0-1.41 0L12 3.23zM1 14.25V19h4.75l9.96-9.96l-4.75-4.75z"/></svg>
\ No newline at end of file
diff --git a/crowdsourcer/templates/crowdsourcer/volunteers/list.html b/crowdsourcer/templates/crowdsourcer/volunteers/list.html
index 62fb554..8fc4248 100644
--- a/crowdsourcer/templates/crowdsourcer/volunteers/list.html
+++ b/crowdsourcer/templates/crowdsourcer/volunteers/list.html
@@ -60,7 +60,7 @@ <h1 class="mb-md-0 me-md-auto">Volunteers</h1>
           {% for volunteer in volunteers %}
             <tr>
                 <td>
-                    <a title="edit user" href="{% session_url 'edit_volunteer' volunteer.pk %}">{{ volunteer.email|default:volunteer.username }}</a>
+                    <a title="Edit user" href="{% session_url 'edit_volunteer' volunteer.pk %}">{{ volunteer.email|default:volunteer.username }}</a>
                 </td>
                 <td>
                     {{ volunteer.marker.response_type|default:"First Mark" }}
@@ -69,8 +69,11 @@ <h1 class="mb-md-0 me-md-auto">Volunteers</h1>
                     {{ volunteer.assigned_section }}
                 </td>
                 <td>
-                    <a title="show progress" href="{% session_url 'volunteer_progress' volunteer.pk %}">{{ volunteer.num_assignments }}</a>
-                    <a title="assign to volunteer" href="{% session_url 'assign_volunteer' volunteer.pk %}">edit</a>
+                    <a title="Show progress" href="{% session_url 'volunteer_progress' volunteer.pk %}">{{ volunteer.num_assignments }}</a>
+                    <a title="Edit assignments" href="{% session_url 'assign_volunteer' volunteer.pk %}" class="ms-1">
+                        {% include 'crowdsourcer/icons/edit.svg' %}
+                        <span class="visually-hidden">Edit</span>
+                    </a>
                 </td>
                 <td>
                     {{ volunteer.is_active }}