Merge pull request #61 from lucaboesch/reddangerdelete

Set delete actions to bottom of kebab menus and tint red, resolves #60.
mebis-lp · Oct 30, 2024 · e5c41c7 · e5c41c7
2 parents 15163ae + d1144ef
commit e5c41c7
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 11 deletions.
diff --git a/templates/actionmenuboard.mustache b/templates/actionmenuboard.mustache
@@ -54,15 +54,15 @@
         <a class="dropdown-item" href="#" data-action="show_template" data-id="{{id}}">{{#pix}} i/publish, core, {{#str}} showtemplate, mod_kanban {{/str}} {{/pix}}{{#str}} showtemplate, mod_kanban {{/str}}</a>
     </li>
     <li class="nav-item mod_kanban_delete_template">
-        <a class="dropdown-item" href="#" data-action="delete_template" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} deletetemplate, mod_kanban {{/str}} {{/pix}}{{#str}} deletetemplate, mod_kanban {{/str}}</a>
+        <a class="dropdown-item text-danger" href="#" data-action="delete_template" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} deletetemplate, mod_kanban {{/str}} {{/pix}}{{#str}} deletetemplate, mod_kanban {{/str}}</a>
     </li>
     {{/manageboard}}
-    <li class="nav-item mod_kanban_delete_board">
-        <a class="dropdown-item" href="#" data-action="delete_board" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} deleteboard, mod_kanban {{/str}} {{/pix}}{{#str}} deleteboard, mod_kanban {{/str}}</a>
-    </li>
     <li class="nav-item mod_kanban_show_board">
         <a class="dropdown-item" href="view.php?id={{cmid}}&group=0&user=0">{{#pix}} t/left, core, {{#str}} showboard, mod_kanban {{/str}} {{/pix}}{{#str}} showboard, mod_kanban {{/str}}</a>
     </li>
+    <li class="nav-item mod_kanban_delete_board">
+        <a class="dropdown-item text-danger" href="#" data-action="delete_board" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} deleteboard, mod_kanban {{/str}} {{/pix}}{{#str}} deleteboard, mod_kanban {{/str}}</a>
+    </li>
     {{#groupmode}}
     {{#groupselector}}
     <li class="nav-item mod_kanban_select_group">
@@ -139,4 +139,4 @@ if (us) {
 }
     {{/viewallboards}}
 {{/userboards}}
-{{/js}}
+{{/js}}
diff --git a/templates/actionmenucard.mustache b/templates/actionmenucard.mustache
@@ -36,9 +36,6 @@
         <li class="nav-item mod_kanban_move_card">
             <a class="dropdown-item" href="#" data-action="move_card" data-id="{{id}}">{{#pix}} i/move_2d, core, {{#str}} move, core {{/str}} {{/pix}}{{#str}} move, core {{/str}}</a>
         </li>
-        <li class="nav-item mod_kanban_delete_card">
-            <a class="dropdown-item" href="#" data-action="delete_card" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} delete, core {{/str}} {{/pix}}{{#str}} delete, core {{/str}}</a>
-        </li>
         <li class="nav-item mod_kanban_complete_card">
             <a class="dropdown-item" href="#" data-action="complete_card" data-id="{{id}}">{{#pix}} t/check, core, {{#str}} complete, core {{/str}} {{/pix}}{{#str}} complete, core {{/str}}</a>
         </li>
@@ -57,5 +54,8 @@
         <li class="nav-item mod_kanban_push_card">
             <a class="dropdown-item" href="#" data-action="push_card" data-id="{{id}}">{{#pix}} t/copy, core, {{#str}} pushcard, mod_kanban {{/str}} {{/pix}}{{#str}} pushcard, mod_kanban {{/str}}</a>
         </li>
+        <li class="nav-item mod_kanban_delete_card">
+            <a class="dropdown-item text-danger" href="#" data-action="delete_card" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} delete, core {{/str}} {{/pix}}{{#str}} delete, core {{/str}}</a>
+        </li>
     </ul>
 </div>
diff --git a/templates/actionmenucolumn.mustache b/templates/actionmenucolumn.mustache
@@ -30,9 +30,6 @@
 <div class="mod_kanban_action_menu{{^managecolumns}} hidden{{/managecolumns}}">
     <button class="mod_kanban_action_menu_trigger btn btn-icon icon-no-margin p-0 dropdown-toggle" data-toggle="dropdown">{{#pix}} i/menu, core, {{#str}} editcolumn, mod_kanban {{/str}} {{/pix}}</button>
     <ul class="dropdown-menu dropdown-menu-right">
-        <li class="nav-item mod_kanban_delete_column">
-            <a class="dropdown-item" href="#" data-action="delete_column" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} delete, core {{/str}} {{/pix}}{{#str}} delete, core {{/str}}</a>
-        </li>
         <li class="nav-item mod_kanban_lock_column">
             <a class="dropdown-item" href="#" data-action="lock_column" data-id="{{id}}">{{#pix}} t/lock, core, {{#str}} lock, mod_kanban {{/str}} {{/pix}}{{#str}} lock, mod_kanban {{/str}}</a>
         </li>
@@ -42,5 +39,8 @@
         <li class="nav-item">
             <a class="dropdown-item" href="#" data-action="edit_details" data-id="{{id}}">{{#pix}} t/edit, core, {{#str}} editdetails, mod_kanban {{/str}} {{/pix}}{{#str}} editdetails, mod_kanban {{/str}}</a>
         </li>
+        <li class="nav-item mod_kanban_delete_column">
+            <a class="dropdown-item text-danger" href="#" data-action="delete_column" data-id="{{id}}">{{#pix}} t/delete, core, {{#str}} delete, core {{/str}} {{/pix}}{{#str}} delete, core {{/str}}</a>
+        </li>
     </ul>
 </div>