From 8d0612b6376fce8fb9a9f0e22d0e983afb1bed89 Mon Sep 17 00:00:00 2001
From: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon, 19 Jul 2021 17:42:24 +0200
Subject: [PATCH] manager: move role delete and permission buttons to kebab
 (#55640)

---
 .../authentic2/manager/role_members.html         | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/authentic2/manager/templates/authentic2/manager/role_members.html b/src/authentic2/manager/templates/authentic2/manager/role_members.html
index 3d693fe9..b6594bf4 100644
--- a/src/authentic2/manager/templates/authentic2/manager/role_members.html
+++ b/src/authentic2/manager/templates/authentic2/manager/role_members.html
@@ -23,11 +23,6 @@
   {{ block.super }}
   <span class="actions">
     <a class="extra-actions-menu-opener"></a>
-  {% if not object.is_internal and view.can_delete %}
-    <a rel="popup" href="{% url "a2-manager-role-delete" pk=object.pk %}">{% trans "Delete" %}</a>
-  {% else %}
-    <a class="disabled" title="{% trans "This role is technical, you cannot delete it." %}" href="#">{% trans "Delete" %}</a>
-  {% endif %}
   {% if view.can_change and not object.is_internal %}
     <a href="{% url "a2-manager-role-edit" pk=object.pk %}">{% trans "Edit" %}</a>
   {% else %}
@@ -36,11 +31,16 @@
     {% else %}
       <a class="disabled" title="{% trans "This role is technical, you cannot edit it." %}" href="#">{% trans "Edit" %}</a>
     {% endif %}
-  {% endif %}
-  {% if perms.a2_rbac.admin_permission %}
-    <a href="{% url "a2-manager-role-permissions" pk=object.pk %}">{% trans "Permissions" %}</a>
   {% endif %}
     <ul class="extra-actions-menu">
+      {% if not object.is_internal and view.can_delete %}
+      <li><a rel="popup" href="{% url "a2-manager-role-delete" pk=object.pk %}">{% trans "Delete" %}</a></li>
+      {% else %}
+      <li><a class="disabled" title="{% trans "This role is technical, you cannot delete it." %}" href="#">{% trans "Delete" %}</a></li>
+      {% endif %}
+      {% if perms.a2_rbac.admin_permission %}
+      <li><a href="{% url "a2-manager-role-permissions" pk=object.pk %}">{% trans "Permissions" %}</a></li>
+      {% endif %}
       <li><a href="{% url "a2-manager-role-journal" pk=object.pk %}">{% trans "Journal" %}</a></li>
     </ul>
   </span>
-- 
2.20.1