From 978a486d8778bab1e7140f0cabf480e80c286d13 Mon Sep 17 00:00:00 2001 From: Christian Soffke Date: Sun, 7 Sep 2025 04:37:29 +0200 Subject: [PATCH] plugins: pictureflow: adjust PF_TRACKLIST behavior 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 --- apps/plugins/pictureflow/pictureflow.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/apps/plugins/pictureflow/pictureflow.c b/apps/plugins/pictureflow/pictureflow.c index fdc4958f4c..ff5584cfd5 100644 --- a/apps/plugins/pictureflow/pictureflow.c +++ b/apps/plugins/pictureflow/pictureflow.c @@ -4936,16 +4936,14 @@ static int pictureflow_main(void) break; #endif case PF_TRACKLIST: - if ( pf_cfg.auto_wps == 1 && pf_state == pf_idle ) { - pf_state = pf_cover_in; - break; - } 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) set_current_slide(target); #if PF_PLAYBACK_CAPABLE - if(pf_cfg.auto_wps == 1) { + if(pf_cfg.auto_wps == 1 && button == PF_SELECT) + { if (start_playback(true)) return PLUGIN_GOTO_WPS; }