2013-11-14 00:53:32 +01:00
|
|
|
commit 6faa447282fe90d42e0513af46c13f20b4b327d4
|
|
|
|
|
Author: Yann E. MORIN <yann.morin.1998@free.fr>
|
|
|
|
|
Date: Wed Nov 13 22:45:02 2013 +0100
|
|
|
|
|
|
|
|
|
|
support/kconfig: fix 'space' to (de)select options
|
|
|
|
|
|
|
|
|
|
In case a menu has comment without letters/numbers (eg. characters
|
|
|
|
|
matching the regexp '^[^[:alpha:][:digit:]]+$', for example - or *),
|
|
|
|
|
hitting space will cycle through those comments, rather than
|
|
|
|
|
selecting/deselecting the currently-highlighted option.
|
|
|
|
|
|
|
|
|
|
This is the behaviour of hitting any letter/digit: jump to the next
|
|
|
|
|
option which prompt starts with that letter. The only letters that
|
|
|
|
|
do not behave as such are 'y' 'm' and 'n'. Prompts that start with
|
|
|
|
|
one of those three letters are instead matched on the first letter
|
|
|
|
|
that is not 'y', 'm' or 'n'.
|
|
|
|
|
|
|
|
|
|
Fix that by treating 'space' as we treat y/m/n, ie. as an action key,
|
|
|
|
|
not as shortcut to jump to prompt.
|
|
|
|
|
|
|
|
|
|
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
|
|
|
|
|
Cc: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
|
|
|
|
|
Cc: Peter Korsgaard <jacmet@uclibc.org>
|
|
|
|
|
Cc: Samuel Martin <s.martin49@gmail.com>
|
|
|
|
|
Cc: Thomas De Schampheleire <patrickdepinguin@gmail.com>
|
|
|
|
|
---
|
|
|
|
|
Note: I'll be running this upstream soonish.
|
|
|
|
|
|
2018-09-19 13:36:15 +02:00
|
|
|
Index: kconfig/lxdialog/menubox.c
|
|
|
|
|
===================================================================
|
|
|
|
|
--- kconfig.orig/lxdialog/menubox.c
|
|
|
|
|
+++ kconfig/lxdialog/menubox.c
|
2013-11-14 00:53:32 +01:00
|
|
|
@@ -285,7 +285,7 @@ do_resize:
|
|
|
|
|
if (key < 256 && isalpha(key))
|
|
|
|
|
key = tolower(key);
|
|
|
|
|
|
|
|
|
|
- if (strchr("ynmh", key))
|
|
|
|
|
+ if (strchr("ynmh ", key))
|
|
|
|
|
i = max_choice;
|
|
|
|
|
else {
|
|
|
|
|
for (i = choice + 1; i < max_choice; i++) {
|
