Skip to content

Improve paused track switching and mini-player swipe controls - #2869

Open
aayushrambo8 wants to merge 2 commits into
PixelPlayerHQ:masterfrom
aayushrambo8:master
Open

aayushrambo8 wants to merge 2 commits into
PixelPlayerHQ:masterfrom
aayushrambo8:master

Conversation

@aayushrambo8

Copy link
Copy Markdown

This PR makes it easier to change tracks and ensures playback resumes appropriately:

  • Resume playback when skipping to the next or previous track while paused, including from the app controls.
  • Preserve previous-track behavior: when more than 10 seconds into a track, Previous restarts the current track without resuming.
  • Change mini-player horizontal swipes from queue dismissal to track navigation: swipe left for next and right for previous, similar to Spotify behaviour.
  • Trigger the track change after a short swipe threshold without moving the mini-player itself.
  • Add unit tests for paused next/previous playback and restarting a track while paused.

Validation: PlaybackStateHolderTest passes, and the mobile debug APK builds successfully.

PS: I have made these changes using AI, but I did frame everything out: how and what system must be followed. This is my 1st public pull, so if you reject it, please tell me what is wrong with it.

Make manual track changes start playback when the player is paused and
replace mini-player swipe-to-dismiss behavior with track navigation.

- Swipe left on the collapsed mini-player to play the next track.
- Swipe right to play the previous track without dismissing or clearing
  the current queue.
- Resume playback after switching tracks from the app controls, widget,
  and Wear OS controls when a destination track is available.
- Preserve previous-track behavior: when the current track has played
  for more than 10 seconds, restart it without resuming playback.
- Add PlaybackStateHolder tests for paused next/previous behavior and
  restarting the current track while paused.
- Reduce Gradle workers from 6 to 2 and configure a 3 GB Kotlin daemon
  heap to reduce compiler memory pressure.
- Remove an unnecessary safe call when reading the non-null album name
  in AI playlist generation.
- Update the Gradle wrapper to 9.7.1.

Validation:
- PlaybackStateHolderTest passes.
- Debug APK build succeeds.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant