Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

enhancement: disable autoplay from AudioPlayer and hides all controls except "Play" at startup #690

Merged
merged 2 commits into from
Dec 26, 2024

Conversation

AkesiSeli
Copy link
Collaborator

@AkesiSeli AkesiSeli commented Dec 26, 2024

Technical details

This PR prevents audio files from automatically start playing and hides all controls except "Play" at startup, as requested in #688.

Additional notes

Unfortunately, there is no simple way to accomplish this using Compose Multiplatform Media Player, so I had to create a "fake" interface of a paused player which shows the real player when the play button is pressed.

@AkesiSeli AkesiSeli self-assigned this Dec 26, 2024
@AkesiSeli AkesiSeli marked this pull request as ready for review December 26, 2024 22:29
@AkesiSeli AkesiSeli marked this pull request as draft December 26, 2024 22:31
@codecov-commenter
Copy link

codecov-commenter commented Dec 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
unittests 58.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@AkesiSeli AkesiSeli linked an issue Dec 26, 2024 that may be closed by this pull request
@AkesiSeli AkesiSeli marked this pull request as ready for review December 26, 2024 22:45
@AkesiSeli AkesiSeli changed the title enhancement: disable autoplay from AudioPlayer enhancement: disable autoplay from AudioPlayer and hides all controls except "Play" at startup Dec 26, 2024
@AkesiSeli AkesiSeli merged commit df0fd2b into master Dec 26, 2024
2 checks passed
@AkesiSeli AkesiSeli deleted the fix-audio-autoplay branch December 26, 2024 22:49
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

More suggestions related to playing audio attachments content
2 participants