summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDana Conrad <dconrad@fastmail.com>2021-03-14 11:27:45 -0500
committerSolomon Peachy <pizza@shaftnet.org>2021-03-14 17:06:22 +0000
commit99383a0f9ba9e940a4017890965494c373a7f09a (patch)
tree46b55dc89e210fbbf24ca28f384ec4853dc4b747
parent6b39e23074afabcc96b9e26ac918534228c48711 (diff)
downloadrockbox-99383a0f9ba9e940a4017890965494c373a7f09a.tar.gz
rockbox-99383a0f9ba9e940a4017890965494c373a7f09a.zip
erosq: change keymap to fix back button
Back button's ACTION_TREE_WPS state in the mainmenu context causes a jump back to the main menu root from anywhere in the settings tree. Changing it to a held button fixes this. Also changed the ACTION_STD_CANCEL functionality of the back button in the standard context to be on release, just because I think it feels better to use. Also added myself to docs/CREDITS Change-Id: Ia5892f4d995431abe0616f00fd5cc8368dd750a2
-rw-r--r--apps/keymaps/keymap-erosq.c6
-rw-r--r--docs/CREDITS3
2 files changed, 6 insertions, 3 deletions
diff --git a/apps/keymaps/keymap-erosq.c b/apps/keymaps/keymap-erosq.c
index 944e5e4ec0..3a841faf82 100644
--- a/apps/keymaps/keymap-erosq.c
+++ b/apps/keymaps/keymap-erosq.c
@@ -39,7 +39,7 @@ static const struct button_mapping button_context_standard[] = {
39 { ACTION_STD_NEXT, BUTTON_NEXT, BUTTON_NONE }, 39 { ACTION_STD_NEXT, BUTTON_NEXT, BUTTON_NONE },
40 { ACTION_STD_NEXTREPEAT, BUTTON_NEXT|BUTTON_REPEAT, BUTTON_NONE }, 40 { ACTION_STD_NEXTREPEAT, BUTTON_NEXT|BUTTON_REPEAT, BUTTON_NONE },
41 { ACTION_STD_OK, BUTTON_PLAY|BUTTON_REL, BUTTON_PLAY }, 41 { ACTION_STD_OK, BUTTON_PLAY|BUTTON_REL, BUTTON_PLAY },
42 { ACTION_STD_CANCEL, BUTTON_BACK, BUTTON_NONE }, 42 { ACTION_STD_CANCEL, BUTTON_BACK|BUTTON_REL, BUTTON_BACK }, // release just because it feels better to use
43 { ACTION_STD_CONTEXT, BUTTON_MENU|BUTTON_REPEAT, BUTTON_MENU }, 43 { ACTION_STD_CONTEXT, BUTTON_MENU|BUTTON_REPEAT, BUTTON_MENU },
44 { ACTION_STD_MENU, BUTTON_MENU|BUTTON_REL, BUTTON_MENU }, 44 { ACTION_STD_MENU, BUTTON_MENU|BUTTON_REL, BUTTON_MENU },
45 { ACTION_STD_KEYLOCK, BUTTON_POWER, BUTTON_NONE }, 45 { ACTION_STD_KEYLOCK, BUTTON_POWER, BUTTON_NONE },
@@ -54,7 +54,9 @@ static const struct button_mapping button_context_standard[] = {
54}; /* button_context_standard */ 54}; /* button_context_standard */
55 55
56static const struct button_mapping button_context_mainmenu[] = { 56static const struct button_mapping button_context_mainmenu[] = {
57 { ACTION_TREE_WPS, BUTTON_BACK, BUTTON_NONE }, 57 { ACTION_TREE_WPS, BUTTON_BACK|BUTTON_REPEAT, BUTTON_BACK }, // settings menu is mainmenu context,
58 // so if we don't use repeat it will
59 // jump back to main menu root
58 60
59 LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_TREE), 61 LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_TREE),
60}; /* button_context_mainmenu as with sansa clip mapping - "back" button returns you to WPS */ 62}; /* button_context_mainmenu as with sansa clip mapping - "back" button returns you to WPS */
diff --git a/docs/CREDITS b/docs/CREDITS
index a8f90c0dd1..720c3a99db 100644
--- a/docs/CREDITS
+++ b/docs/CREDITS
@@ -743,4 +743,5 @@ The Pocket Quake team (Dan East and others)
743The bzip2 team 743The bzip2 team
744The bsdiff team 744The bsdiff team
745The libtomcrypt team 745The libtomcrypt team
746Spencer Brennessel \ No newline at end of file 746Spencer Brennessel
747Dana Conrad