-
Notifications
You must be signed in to change notification settings - Fork 175
Demo doesn't work on Safari 7.1 #13
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
Comments
@kingsfleet I just pushed a fix - great suggestion. Could you try it again and let me know if it is working for you? |
Odd still serving the old version. Is there something else that needs to be done? On 9 Oct 2014, at 19:55, Zach Pomerantz [email protected] wrote:
|
Yes! It needs to be changed in the demo, which is on a separate branch. Separately, a build process needs to be put in place to keep the two branches in sync. |
I'd be grateful if others could test this and make sure it works cross-system. I made a basic build script (run by Travis) that should avoid this issue in the future. |
Hi, It then goes on to fail with safari as getUserMedia isn't supported: http://caniuse.com/#feat=stream Is there an AudioContext API you could use instead? It might make it also work on IE. |
I'm not sure of another API to capture microphone intput, besides I built this as a toy project, and use Chrome myself, so I hadn't considered compatibility to a large extent. @kingsfleet I would love if you could PR a shim for unsupported browsers. |
Apparently there is a webkitGetUserMedia, can you try that and I will see if the demo works? (Sorry for the delay in getting back to you.) |
I can try it later today when I have more time. For reference, this is where I usually check for compatibity, followed by mdn. I'm sure there are Flash shims that polyfill the microphone input feature, but I haven't looked into them yet, aside from seeing that they exist. That might be the alternative that needs to be pursued in this case. |
webAudioContext is there but not AudioContext, could the constructor code be replaced with this:
var AudioContext = window.AudioContext || window.webkitAudioContext;
audio.contex = new AudioContext();
The text was updated successfully, but these errors were encountered: