[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rockbox-cvs
Subject:    iPod: Fix Yes/No Keymap
From:       rockbox-gerrit-noreply--- via rockbox-cvs <rockbox-cvs () lists ! haxx ! se>
Date:       2021-08-31 18:59:45
Message-ID: 202108311859.17VIxj6D588965 () stuffed ! shaftnet ! org
[Download RAW message or body]

commit d929444a413007244155e88bf87e7c75f0d2da30
Author: Dana Conrad <dconrad@fastmail.com>
Date:   Mon Aug 23 18:43:25 2021 -0500

    iPod: Fix Yes/No Keymap
    
    Add fallback to CONTEXT_STD and add explicit Play button cancel -
    Play button is not present in standard context, so it won't do
    anything if unknown actions are ignored.
    
    tested on ipod simulator
    
    Change-Id: Ic259fa97b6195ebbc66fcfae31265cc7aeefa421

diff --git a/apps/keymaps/keymap-ipod.c b/apps/keymaps/keymap-ipod.c
index ca52b6b216..e3a17fffba 100644
--- a/apps/keymaps/keymap-ipod.c
+++ b/apps/keymaps/keymap-ipod.c
@@ -114,7 +114,8 @@ static const struct button_mapping button_context_settings[]  = {
 
 static const struct button_mapping button_context_yesno[]  = {
     { ACTION_YESNO_ACCEPT,          BUTTON_SELECT,                  BUTTON_NONE },
-    LAST_ITEM_IN_LIST
+    { ACTION_STD_CANCEL,          BUTTON_PLAY,                  BUTTON_NONE },
+    LAST_ITEM_IN_LIST__NEXTLIST(CONTEXT_STD)
 }; /* button_context_yesno */
 
 static const struct button_mapping button_context_bmark[]  = {
-- 
rockbox-cvs mailing list
rockbox-cvs@lists.haxx.se
https://lists.haxx.se/listinfo/rockbox-cvs
[prev in list] [next in list] [prev in thread] [next in thread]