Fix iOS control center notification in html5 mode #1530
Conversation
When html5 is forced, we don't really need to setupAudioContext. When it's called too early, it unnecessarily creates a Web Audio gain node which breaks the notification on iOS.
Linux/GNU sed can't handle `-i ''` ("in-place editing with empty filename)
omitting it reads from stdin and works on macOS (BSD) just as well.
|
Any chance of getting this merged? |
|
I'm not able to merge this due to the distribution files being modified. However, I'm still seeing issues with the control center with these changes. Still doing some digging. |
|
No worries! I need to get around to setting up the guidelines on that through GitHub's contributor features. The build happens when when a new npm version goes out, so no need to do that in the PR. It might be a separate issue, but if you pause in the control center and then play again the audio can't be heard until you play directly through howler again. |
|
@bikubi Thanks for putting up this PR and for your comments on the related issues. I was excited to discover this PR, and I found that it gets the media back into the control center! However, I'm running into the same bug that @goldfire describes:
Your comment on #1262 appears to be related, but that's not fixing the issue for me. It appears to me that once the audio is paused, the MediaSession isn't firing the |
|
@goldfire I put together a simple player page that can be used to test the mobile lock screen and control center controls. Once I added in code to configure and handle Can you advise on how best to put up a PR for you to review? One path would be to put up a PR with just the tests, which demonstrate that the controls are not working in the main branch. And then if you rebase this PR onto that, you'll see that they are. Another path would be to put up a PR against this PR branch, but it's not clear to me that I have permission to do that in this repo. |
|
@goldfire the issues with iOS is still a bother. Can we merge the fix? |
Hi. I had forked this lib and merged a few urgent PRs. If you want, make a pr to https://github.com/profispojka/loudest and you will soon find the changes in https://www.npmjs.com/package/loudest |
|
Is there any news on merging this @goldfire ? 🤞 I am using the use-sound library which uses howler and have no idea if and how I could fix this manually myself else I would. Even if it is not a full fix, it does not feel like it breaks anything and is a nice addition? |
This is a fix for #1383. See commit message for how it works.
I've ran all tests & examples successfully.
I've also fixed to build script to work on Linux. It still works on my Mac, too (current macOS, sed from 2017 -- should be fine, I guess).