Add play actions for playlists - #506
Open
Padma-Dev-E wants to merge 2 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds playlist-level Play actions so users can start playback directly from playlist headers instead of playing tracks one by one.
This affects:
The Play action starts only available local/downloaded tracks in playlist order, disables shuffle before playback, and shows an error snackbar if no playable local file is available.
Related Issues
Fixes #490
Type of Change
Validation
dart format --output=none --set-exit-if-changed lib testflutter analyzeflutter testgofmt,go vet ./..., andgo test ./...ingo_backend/Validation notes:
Verified on iPhone 17 simulator. Confirmed the playlist header now shows Play next to Download All, and tapping Play starts playback for available local/downloaded playlist tracks in order.
Skipped
flutter testbecause this change is UI/playback wiring and no targeted widget/unit test exists for this playlist header flow.Skipped Go checks because this PR does not change
go_backend/.Contributor Checklist
Screenshots or Recordings
Add screenshot/video showing the playlist header with the new Play button next to Download All.
