Tidier "Edit" action for assignments on volunteer list

mysociety · Nov 21, 2024 · 723f343 · 723f343
1 parent 5eba8a5
commit 723f343
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/crowdsourcer/templates/crowdsourcer/icons/edit.svg b/crowdsourcer/templates/crowdsourcer/icons/edit.svg
diff --git 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 }}