Skip to content

Add support for volume control and mute state toggling#962

Merged
MGaetan89 merged 2 commits into928-custom-castplayerfrom
935-volume-management
Mar 28, 2025
Merged

Add support for volume control and mute state toggling#962
MGaetan89 merged 2 commits into928-custom-castplayerfrom
935-volume-management

Conversation

@MGaetan89
Copy link
Contributor

@MGaetan89 MGaetan89 commented Mar 27, 2025

Pull request

Description

This commit adds support for volume management (local and device volume), as well as toggling the muted state of the device.

Changes made

  • Add support for volume control, device volume control, and muted state toggling to PillarboxCastPlayer.
  • Update pillarbox-demo-cast to add a mute button and volume slider.
  • Push the automatic changes to .idea/codeStyles/Project.xml.

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 Mar 27, 2025
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Pillarbox Mar 27, 2025
@MGaetan89 MGaetan89 linked an issue Mar 27, 2025 that may be closed by this pull request
Scaffold(modifier = Modifier.fillMaxSize()) { innerPadding ->
Box(modifier = Modifier.padding(innerPadding)) {
MainView(player = player, modifier = Modifier)
MainView(player)

Check warning

Code scanning / Android Lint

Synthetic Accessor Warning

Access to private method MainView of class MainActivityKt requires synthetic accessor
@MGaetan89 MGaetan89 force-pushed the 935-volume-management branch from 931fafe to c237481 Compare March 27, 2025 07:06
@github-actions
Copy link

github-actions bot commented Mar 27, 2025

Code Coverage

Overall Project 49.82% -0.69% 🟢
Files changed 22.03% 🟢

Module Coverage
:pillarbox-player 56.97% -0.38% 🟢
:pillarbox-cast 7.99% -6.28% 🟢
:pillarbox-ui 0.27% -1.23% 🟢
Files
Module File Coverage
:pillarbox-player PlayerCallbackFlow.kt 67.49% -4.3% 🟢
:pillarbox-cast PillarboxCastPlayer.kt 4% -9.15% 🟢
:pillarbox-ui ComposablePlayer.kt 0% -7.23% 🟢

@MGaetan89 MGaetan89 requested a review from StaehliJ March 27, 2025 07:11
Base automatically changed from RemoteMediaClient_nullability to 928-custom-castplayer March 27, 2025 13:17
@MGaetan89 MGaetan89 force-pushed the 935-volume-management branch from c237481 to 226af09 Compare March 27, 2025 13:26
@MGaetan89 MGaetan89 merged commit 709c2b6 into 928-custom-castplayer Mar 28, 2025
9 of 10 checks passed
@MGaetan89 MGaetan89 deleted the 935-volume-management branch March 28, 2025 16:10
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Pillarbox Mar 28, 2025
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.

Volume management

2 participants