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

Demo doesn't work on Safari 7.1 #13

Open
kingsfleet opened this issue Oct 9, 2014 · 8 comments
Open

Demo doesn't work on Safari 7.1 #13

kingsfleet opened this issue Oct 9, 2014 · 8 comments
Labels

Comments

@kingsfleet
Copy link

webAudioContext is there but not AudioContext, could the constructor code be replaced with this:

var AudioContext = window.AudioContext || window.webkitAudioContext;
audio.contex = new AudioContext();

@zzmp
Copy link
Owner

zzmp commented Oct 9, 2014

@kingsfleet I just pushed a fix - great suggestion. Could you try it again and let me know if it is working for you?

@zzmp zzmp closed this as completed Oct 9, 2014
@zzmp zzmp added the bug label Oct 9, 2014
@kingsfleet
Copy link
Author

Odd still serving the old version. Is there something else that needs to be done?

On 9 Oct 2014, at 19:55, Zach Pomerantz notifications@github.com wrote:

@kingsfleet I just pushed a fix - great suggestion. Could you try it again and let me know if it is working for you?


Reply to this email directly or view it on GitHub.

@zzmp
Copy link
Owner

zzmp commented Oct 10, 2014

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.

@zzmp zzmp reopened this Oct 10, 2014
@zzmp
Copy link
Owner

zzmp commented Oct 15, 2014

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.

@kingsfleet
Copy link
Author

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.

@zzmp
Copy link
Owner

zzmp commented Oct 17, 2014

I'm not sure of another API to capture microphone intput, besides getUserMedia. Do you know of a shim to work on Safari/IE?

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.

@kingsfleet
Copy link
Author

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.)

@zzmp
Copy link
Owner

zzmp commented Oct 26, 2014

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants