Skip to content

Use Media3 states for shuffle and repeat in :pillarbox-demo#1133

Merged
MGaetan89 merged 4 commits intomainfrom
1112-improve-player-demo_shuffle-repeat
Aug 11, 2025
Merged

Use Media3 states for shuffle and repeat in :pillarbox-demo#1133
MGaetan89 merged 4 commits intomainfrom
1112-improve-player-demo_shuffle-repeat

Conversation

@MGaetan89
Copy link
Contributor

@MGaetan89 MGaetan89 commented Aug 7, 2025

Pull request

Description

This PR integrates Media3's RepeatButtonState and ShuffleButtonState to manage the corresponding buttons in :pillarbox-demo.

Changes made

  • Use RepeatButtonState and ShuffleButtonState to manage the repeat and shuffle buttons in :pillarbox-demo.
  • Split PlayerBottomToolbar into smaller components for readability.
  • Deprecate Player.shuffleModeEnabledAsState() and Player.repeatModeAsState() in favor of the Media3 solution.

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@MGaetan89 MGaetan89 self-assigned this Aug 7, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Aug 7, 2025
@MGaetan89 MGaetan89 changed the title Integrate rememberShuffleButtonState() from Media3 Use Media3 states for shuffle and repeat in :pillarbox-demo Aug 7, 2025
@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Code Coverage

Overall Project 49.16% 🟢
Module Coverage
:pillarbox-ui 0.26% 🟢
Files
Module File Coverage
:pillarbox-ui ComposablePlayer.kt 0% 🟢

@MGaetan89 MGaetan89 marked this pull request as ready for review August 7, 2025 13:06
@MGaetan89 MGaetan89 requested a review from StaehliJ August 7, 2025 13:06
@MGaetan89 MGaetan89 moved this from 📋 Backlog to 🍿 Code Review in Pillarbox Aug 7, 2025
@MGaetan89 MGaetan89 force-pushed the 1112-improve-player-demo_shuffle-repeat branch from a5309db to adb0f58 Compare August 11, 2025 08:27
@MGaetan89 MGaetan89 added this pull request to the merge queue Aug 11, 2025
Merged via the queue into main with commit 67edee7 Aug 11, 2025
12 checks passed
@MGaetan89 MGaetan89 deleted the 1112-improve-player-demo_shuffle-repeat branch August 11, 2025 09:38
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox Aug 11, 2025
@MGaetan89 MGaetan89 linked an issue Aug 11, 2025 that may be closed by this pull request
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Improve Player demo

2 participants