From 618beee24166caf02f2bc881a1d27a4ef1b3ccbd Mon Sep 17 00:00:00 2001
From: Daniel Friesel <derf@derf.homelinux.org>
Date: Mon, 7 Jun 2010 19:59:09 +0200
Subject: Removed undocumented hjkl menu keys

---
 ChangeLog       |  3 +++
 src/keyevents.c | 24 +-----------------------
 2 files changed, 4 insertions(+), 23 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 0e2f1f3..1d66282 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -7,6 +7,9 @@ git HEAD
       anyways. Plus, I might want to bind Shift + <already used key> to other
       stuff in the future
     * Documentation improvements
+    * Remove undocumented hjkl menu keys. Most of the menu options can also be
+      accessed directly with keys, plus the number of non-qwerty users (which
+      have no use for these shortcuts) is growing.
 
 Sat Jun  5 21:35:25 CEST 2010  Daniel Friesel <derf@chaosdorf.de
 
diff --git a/src/keyevents.c b/src/keyevents.c
index e513775..a5cafb9 100644
--- a/src/keyevents.c
+++ b/src/keyevents.c
@@ -93,29 +93,7 @@ void feh_event_handle_keypress(XEvent * ev)
 			feh_menu_select_submenu(selected_menu);
 			break;
 		case XK_Return:
-			feh_menu_item_activate(selected_menu, selected_item);
-			break;
-		default:
-			break;
-		}
-		if (len <= 0 || len > (int) sizeof(kbuf))
-			D_RETURN_(4);
-		kbuf[len] = '\0';
-
-		switch (*kbuf) {
-		case 'h':
-			feh_menu_select_parent(selected_menu);
-			break;
-		case 'j':
-			feh_menu_select_next(selected_menu, selected_item);
-			break;
-		case 'k':
-			feh_menu_select_prev(selected_menu, selected_item);
-			break;
-		case 'l':
-			feh_menu_select_submenu(selected_menu);
-			break;
-		case ' ':
+		case XK_space:
 			feh_menu_item_activate(selected_menu, selected_item);
 			break;
 		default:
-- 
cgit v1.2.3