Delete key doesn't open up end process dialogue when pressed on selected process [Feature Request]

[Yetoo1]

Expected behaviour

Pressing the delete key should make the end process dialogue box appear.

Actual behaviour

Nothing happens at all when the delete key is pressed.

Steps to reproduce the behaviour

Press the delete key on a selected process.

MATE general version

1.18.0-1 Ubuntu

Package version

1.19.0 built from the master branch.

Linux Distribution

Ubuntu 17.10 Developmental release

gnome-system-manager has this feature yet this doesn't. I looked through the source code and I didn't find a single binding for the delete key. This would be convenient for those who are used to the keyboard short cut paradigm involving the delete key causing the deletion of a selection.

---

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[Yetoo1]

In case I gave actual valid reasons, I added the feature. I'm not sure how to add a patch with git so I'll just paste it here.

From 1198265841e2560f878341414c6019aa29a151d8 Mon Sep 17 00:00:00 2001
From: Yetoo1 <[email protected]>
Date: Wed, 11 Oct 2017 09:59:58 -0700
Subject: [PATCH] interface: Allows Delete key to kill processes

The delete key now can kill selected processes as Ctrl-K can while
the menu entry displaying the Ctrl+K shortcut remains. The purpose of
this is to allow convenience for those who are switching over from
gnome-system-monitor or other process managers which allow the
delete key to kill selected processes. This was accomplished by
adding an accelerator item with the delete key as the accelerator
and putting the menu entry of the KillProcess_Del on top of the
KillProcess menu entry so the latter shows over the former.

Issue #101
---
 src/interface.cpp | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/interface.cpp b/src/interface.cpp
index bacab98..3e41a62 100644
--- a/src/interface.cpp
+++ b/src/interface.cpp
@@ -66,6 +66,8 @@ static const GtkActionEntry menu_entries[] =
 
     { "EndProcess", NULL, N_("_End Process"), "<control>E",
       N_("Force process to finish normally"), G_CALLBACK (cb_end_process) },
+    { "KillProcess_Del", NULL, N_("_Kill Process"), "Delete",
+      N_("Force process to finish immediately"), G_CALLBACK (cb_kill_process) },
     { "KillProcess", NULL, N_("_Kill Process"), "<control>K",
       N_("Force process to finish immediately"), G_CALLBACK (cb_kill_process) },
     { "ChangePriority", NULL, N_("_Change Priority"), NULL,
@@ -137,6 +139,7 @@ static const char ui_info[] =
     "      <menuitem name=\"EditContProcessMenu\" action=\"ContProcess\" />"
     "      <separator />"
     "      <menuitem name=\"EditEndProcessMenu\" action=\"EndProcess\" />"
+    "      <menuitem name=\"EditKillProcessMenu\" action=\"KillProcess_Del\" />"
     "      <menuitem name=\"EditKillProcessMenu\" action=\"KillProcess\" />"
     "      <separator />"
     "      <menu name=\"EditChangePriorityMenu\" action=\"ChangePriority\" >"
@@ -175,6 +178,7 @@ static const char ui_info[] =
     "    <menuitem action=\"ContProcess\" />"
     "    <separator />"
     "    <menuitem action=\"EndProcess\" />"
+    "    <menuitem action=\"KillProcess_Del\" />"
     "    <menuitem action=\"KillProcess\" />"
     "    <separator />"
     "    <menu name=\"ChangePriorityMenu\" action=\"ChangePriority\" >"
@@ -835,6 +839,7 @@ update_sensitivity(ProcData *data)
     const char * const selected_actions[] = { "StopProcess",
                                               "ContProcess",
                                               "EndProcess",
+                                              "KillProcess_Del"
                                               "KillProcess",
                                               "ChangePriority",
                                               "MemoryMaps",
-- 
2.14.1

[tylnesh]

AFAIK, you should fork the repository, make the changes and submit a pull request. If you can't be bothered, I'll do it for you, as I'd like to see this feature added as well.