feat: improve button responsiveness and track navigation seek options

- Reduce button debounce interval for faster responsiveness
- Support 5-minute seeking using track navigation buttons on single long tracks

Author: fhirschmann

src/AudioPlayer.cpp

@@ -758,9 +758,16 @@ void AudioPlayer_Loop() {
 						audio->stopSong();
 					}
 				} else {
-					Log_Println(lastTrackAlreadyActive, LOGLEVEL_NOTICE);
-					System_IndicateError();
-					return;
+					if (gPlayProperties.playlist->size() == 1 && audio->getAudioFileDuration() > 300) {
+						audio->setTimeOffset(300);
+						Log_Printf(LOGLEVEL_NOTICE, secondsJumpForward, 300);
+						trackCommand = NO_ACTION;
+						return;
+					} else {
+						Log_Println(lastTrackAlreadyActive, LOGLEVEL_NOTICE);
+						System_IndicateError();
+						return;
+					}
 				}
 				break;
 
@@ -793,12 +800,10 @@ void AudioPlayer_Loop() {
 					}
 				} else {
 					if (gPlayProperties.currentTrackNumber > 0 || gPlayProperties.repeatPlaylist) {
-						if (audio->getAudioCurrentTime() < 5) { // play previous track when current track time is small, else play current track again
-							if (gPlayProperties.currentTrackNumber == 0 && gPlayProperties.repeatPlaylist) {
-								gPlayProperties.currentTrackNumber = gPlayProperties.playlist->size() - 1; // Go back to last track in loop-mode when first track is played
-							} else {
-								gPlayProperties.currentTrackNumber--;
-							}
+						if (gPlayProperties.currentTrackNumber == 0 && gPlayProperties.repeatPlaylist) {
+							gPlayProperties.currentTrackNumber = gPlayProperties.playlist->size() - 1; // Go back to last track in loop-mode when first track is played
+						} else {
+							gPlayProperties.currentTrackNumber--;
 						}
 
 						if (gPlayProperties.saveLastPlayPosition) {
@@ -811,6 +816,12 @@ void AudioPlayer_Loop() {
 							audio->stopSong();
 						}
 					} else {
+						if (gPlayProperties.playlist->size() == 1 && audio->getAudioFileDuration() > 300) {
+							audio->setTimeOffset(-300);
+							Log_Printf(LOGLEVEL_NOTICE, secondsJumpBackward, 300);
+							trackCommand = NO_ACTION;
+							return;
+						}
 						if (gPlayProperties.saveLastPlayPosition) {
 							AudioPlayer_NvsRfidWriteWrapper(gPlayProperties.playRfidTag, 0, gPlayProperties.playMode, gPlayProperties.currentTrackNumber);
 						}

src/Button.cpp

@@ -215,8 +215,8 @@ static void Button_UpdateState(uint8_t i, t_button &btn, unsigned long currentTi
 			btn.lastReleasedTimestamp = currentTimestamp;
 			btn.firstPressedTimestamp = 0;
 		}
+		btn.lastState = btn.currentState;
 	}
-	btn.lastState = btn.currentState;
 }
 
 // If timer-semaphore is set, read buttons (unless controls are locked)

src/settings-override.h.sample

@@ -159,7 +159,7 @@
 	#endif
 
 	// Buttons (better leave unchanged if in doubts :-))
-	constexpr uint8_t buttonDebounceInterval = 50;                // Interval in ms to software-debounce buttons
+	constexpr uint8_t buttonDebounceInterval = 30;                // Interval in ms to software-debounce buttons
 	constexpr uint16_t intervalToLongPress = 700;                 // Interval in ms to distinguish between short and long press of buttons
 
 	// Buttons active state: Default 0 for active LOW, 1 for active HIGH e.g. for TTP223 Capacitive Touch Switch Button (FinnBox)

src/settings.h

@@ -167,7 +167,7 @@
 	#endif
 
 	// Buttons (better leave unchanged if in doubts :-))
-	constexpr uint8_t buttonDebounceInterval = 50;                // Interval in ms to software-debounce buttons
+	constexpr uint8_t buttonDebounceInterval = 30;                // Interval in ms to software-debounce buttons
 	constexpr uint16_t intervalToLongPress = 700;                 // Interval in ms to distinguish between short and long press of buttons
 
 	// Buttons active state: Default 0 for active LOW, 1 for active HIGH e.g. for TTP223 Capacitive Touch Switch Button (FinnBox)