Not rendering on iOS or macOS Safari about vue-audio-visual

[staskobzar]

Would you mind replacing the wontfix label with help wanted?
(I think that) the wontfix label sends a signal that you have no intention of fixing this and it works as expected, whereas help wanted shows you're willing to have this fixed, but don't have the resources for it yourself atm.

good point. changed. thanks

[andreasvirkus]

Would you mind replacing the wontfix label with help wanted?
(I think that) the wontfix label sends a signal that you have no intention of fixing this and it works as expected, whereas help wanted shows you're willing to have this fixed, but don't have the resources for it yourself atm.

[staskobzar]

Hello,
It also may depend on the <audio\> element. What attributes of the audio element do you use?
Also, does it works without v-show condition?

[szokeptr]

My audio tag looks like this:

<audio 
    ref="player" 
    :src="audioSource" 
    :title="radio.now_playing.song.title" 
    crossorigin="anonymous"
></audio>

audioSource is a streaming mp3 file, title is a plain string with the song title.

Also, it's not working without the v-show condition either (I removed the directive entirely).

[szokeptr]

Quick note: I tried pulling the canvas contents with the toDataURL method and it appears to be a static un-styled blue circle all the time.

[staskobzar]

I can not find the way to fix it for safari. Spent whole day playing with different parameters and re-writing code trying to fix it.
I am sorry but I have to leave it like this until I find any other way to fix it or may be Apple will do something with Safari. May be someone will contribute a fix.

[andreipopovici]

For macOS Safari at least, the AudioContext needs to be created on "user interaction" i.e. inside the context of a click event. This would require some significant refactoring. Both the audio element that gets created and the one attached via refLink need an onclick listener, and setAnalyser needs to be called inside that listener, not in mount().

I'll play around and see if I can fix this and open a PR.

[staskobzar]

@andreipopovici thank you. Your patch works great with my tests (macOS/Safari 13.1). Just merged. Will be released in ver 2.0.1

[andreipopovici]

After spending way too many hours, I realized why the fix in #33 was not working on mobile Safari. It's a bug that has ben acknowledged but not fixed yet: https://bugs.webkit.org/show_bug.cgi?id=203435

Nothing more to do here 🤷‍♂️

[staskobzar]

Good, thank you @andreasvirkus
This one took me many hours too but I could not find it. I do appreciate your contribution.

I'm closing this ticket. Have a good week-end!

[andreipopovici]

It looks like that bug was fixed via https://bugs.webkit.org/show_bug.cgi?id=211394 in iOS 13.6 upward, but I didn't have a chance to test and see if it's fixed this particular issue. Hopefully it did!

[staskobzar]

Good! hopefully it does.
Thank you