Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

mute the video #45

Closed
wants to merge 2 commits into from
Closed

mute the video #45

wants to merge 2 commits into from

Conversation

greggman
Copy link
Contributor

At least on iOS this works and has the benefit the user's
music doesn't stop playing yet it still doesn't sleep.

I don't have Android to test on

At least on iOS this works and has the benefit the user's
music doesn't stop playing yet it still doesn't sleep
@richtr
Copy link
Owner

richtr commented Mar 15, 2018

I'm going to test this now on both Android and iOS devices!

@richtr
Copy link
Owner

richtr commented Mar 15, 2018

According to #47 this does not actually mute the video on Android devices. We might want to keep the idea provided in this PR + include a separate video src for Android devices that does not include an audio track (only iOS Safari requires an audio track in the video to enable wake lock).

@montamal
Copy link

I have a similar solution but I unmute the video inside the enable method before playing it so that it works on both iOS and Android.

@richtr
Copy link
Owner

richtr commented Dec 4, 2018

FYI, commit 615ee56 from this PR has now been cherry-picked on to this PR: #58 (branch name: feature/webm-mp4-dual-video)

@richtr
Copy link
Owner

richtr commented Dec 4, 2018

Merged as part of #58.

Thank you for this! 👍

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.

None yet

3 participants