devs/foxbox
1
0
Fork 0
forked from len0rd/rockbox
Code Pull requests Activity

plugins: pictureflow: adjust PF_TRACKLIST behavior

Browse source 
Clarifies presumably intentional fall-through for
PF_TRACKLIST and fixes unwanted behavior of
PF_TRACKLIST while scrolling, which may have
started playback instead of showing the track list.

(affected keymaps at the moment are only SANSA_FUZE_PAD,
SAMSUNG_YH820_PAD, SAMSUNG_YH92X_PAD, and FIIO_M3K_PAD)

Change-Id: I335dd1b4a2152caed966b73aaa189bcfddc79734
This commit is contained in:
Christian Soffke 2025-09-07 04:37:29 +02:00
parent 34f34bb7f1
commit 978a486d87
1 changed files with 4 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

10
apps/plugins/pictureflow/pictureflow.c
View file

@ -4936,16 +4936,14 @@ static int pictureflow_main(void)
break; break;
#endif #endif
case PF_TRACKLIST: case PF_TRACKLIST:
if ( pf_cfg.auto_wps == 1 && pf_state == pf_idle ) {
pf_state = pf_cover_in;
break;
}
case PF_SELECT: case PF_SELECT:
if ( pf_state == pf_idle || pf_state == pf_scrolling) { if (pf_state == pf_idle || pf_state == pf_scrolling)
{
if (pf_state == pf_scrolling) if (pf_state == pf_scrolling)
set_current_slide(target); set_current_slide(target);
#if PF_PLAYBACK_CAPABLE #if PF_PLAYBACK_CAPABLE
if(pf_cfg.auto_wps == 1) { if(pf_cfg.auto_wps == 1 && button == PF_SELECT)
{
if (start_playback(true)) if (start_playback(true))
return PLUGIN_GOTO_WPS; return PLUGIN_GOTO_WPS;
} }