NEWS

@@ -5,8 +5,6 @@ Unreleased
 
  * Improved song information shown in the window header
 
- * Escape no longer quits the program
-
  * X11: added italic font support
 
  * X11: fixed rendering of overflowing, partially visible list items

nncmpp.actions

@@ -2,7 +2,6 @@ NONE,               Do nothing
 
 QUIT,               Quit
 REDRAW,             Redraw screen
-ABORT,              Abort
 TAB_HELP,           Switch to help tab
 TAB_LAST,           Switch to last tab
 TAB_PREVIOUS,       Switch to previous tab

nncmpp.c

@@ -2662,14 +2662,6 @@ app_process_action (enum action action)
 	case ACTION_NONE:
 		return true;
 	case ACTION_QUIT:
-		app_quit ();
-		return true;
-	case ACTION_REDRAW:
-		clear ();
-		app_invalidate ();
-		return true;
-
-	case ACTION_ABORT:
 		// It is a pseudomode, avoid surprising the user
 		if (tab->item_mark > -1)
 		{
@@ -2677,7 +2669,13 @@ app_process_action (enum action action)
 			app_invalidate ();
 			return true;
 		}
-		return false;
+
+		app_quit ();
+		return true;
+	case ACTION_REDRAW:
+		clear ();
+		app_invalidate ();
+		return true;
 	case ACTION_MPD_COMMAND:
 		line_editor_start (&g.editor, ':');
 		g.editor.on_end = app_on_mpd_command_editor_end;
@@ -2809,7 +2807,7 @@ app_editor_process_action (enum action action)
 	app_invalidate ();
 	switch (action)
 	{
-	case ACTION_ABORT:
+	case ACTION_QUIT:
 		line_editor_abort (&g.editor, false);
 		g.editor.on_end = NULL;
 		return true;
@@ -3009,9 +3007,9 @@ static struct binding_default
 }
 g_normal_defaults[] =
 {
+	{ "Escape",     ACTION_QUIT               },
 	{ "q",          ACTION_QUIT               },
 	{ "C-l",        ACTION_REDRAW             },
-	{ "Escape",     ACTION_ABORT              },
 	{ "M-Tab",      ACTION_TAB_LAST           },
 	{ "F1",         ACTION_TAB_HELP           },
 	{ "S-Tab",      ACTION_TAB_PREVIOUS       },
@@ -3077,10 +3075,6 @@ g_normal_defaults[] =
 },
 g_editor_defaults[] =
 {
-	{ "C-g",        ACTION_ABORT              },
-	{ "Escape",     ACTION_ABORT              },
-	{ "Enter",      ACTION_EDITOR_CONFIRM     },
-
 	{ "Left",       ACTION_EDITOR_B_CHAR      },
 	{ "Right",      ACTION_EDITOR_F_CHAR      },
 	{ "C-b",        ACTION_EDITOR_B_CHAR      },
@@ -3104,6 +3098,10 @@ g_editor_defaults[] =
 	{ "C-u",        ACTION_EDITOR_B_KILL_LINE },
 	{ "C-k",        ACTION_EDITOR_F_KILL_LINE },
 	{ "C-w",        ACTION_EDITOR_B_KILL_WORD },
+
+	{ "C-g",        ACTION_QUIT               },
+	{ "Escape",     ACTION_QUIT               },
+	{ "Enter",      ACTION_EDITOR_CONFIRM     },
 };
 
 static int
@@ -6750,10 +6748,7 @@ on_x11_event (XEvent *ev)
 	case SelectionClear:
 		cstr_set (&g.x11_selection, NULL);
 		break;
-	// UnmapNotify can be received when restarting the window manager.
-	// Should this turn out to be unreliable (window not destroyed by WM
-	// upon closing), opt for the WM_DELETE_WINDOW protocol as well.
-	case DestroyNotify:
+	case UnmapNotify:
 		app_quit ();
 		break;
 	case FocusIn: