-
Notifications
You must be signed in to change notification settings - Fork 61
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
IE9 and IE10 support #37
Comments
Hi, can you let me know which Phono version gives you this error? |
The latest one: 0.5 seems to work fine. At least it loads fine. |
Would it be possible for you to try the current master on GitHub to see if we have already resolved the issue or if it still happens? |
Yeah, ok. I will do this later. |
Ok, I've tried a nightly build.
This happens not only my local machine, but on staging server too: |
Many thanks - can you let me know if you are using https to serve the host page? |
Not on staging, but production does use https. http://bizzocall2dev.aws.af.cm/endpoint/static/id/36 (hit the connect via internet) |
By the way, despite the fact that kitchen sink on version 0.5 initializes in IE9 and IE 10, when I try to make an actual call I get the same error that's been there since version 0.3:
|
I believe that this may be IE throwing the "SCRIPT5: Access is denied" error if the whole page and connection back to Phono isn't using SSL. Do you have a way to test SSL with a valid certificate on your staging network? (Thanks for helping to debug this.) |
Here you go: Same 'Access denied' error. |
I've made some changes to the master branch. Please can you retest the current master and let me know if that helps? |
Well, the good news is that kitchen sink demo works in IE8, IE9 and IE10. Here's the log from IE10: You can test here: I load phono code via ajax. Could this be the issue? |
Yes, got this too! Maybe Strophe bug? |
I think I figured out the problem on IE9. It because the |
Do you still get this error when using the master branch from github? |
I just check the master branch, there's still some errors in IE. I have to monkey patch the Phono.util = {
//...
loggyFunction: function(objName, obj, funcName) {
var original = obj[funcName];
obj[funcName] = function() {
// Convert arguments to a real array
var sep = "";
var args = "";
for (var i = 0; i < arguments.length; i++) {
try {
args+= (sep + arguments[i]);
} catch(e) {
// To fix bug in IE when concatinating an object to a string.
args+= (sep + typeof(arguments[i]));
}
sep = ",";
}
Phono.log.debug("[INVOKE] " + objName + "." + funcName + "(" + args + ")");
return original.apply(obj, arguments);
}
} After that it works fine in IE8 and IE10, but not in IE9 :D
the error is:
Maybe error in the native flash Audio class? |
I found the problem! It's because the microphone is null in IE9. I don't know why, but I try to fix and make it work in IE9 now. |
Updating to phono.js 1.0 fixed all my issues in IE. IE7 - IE10 now works. Thanks! |
Hi,
Kitchen sink demo doesn't work in my IE10, if I choose IE9 or IE10 mode, Nor it works in quircks.
IE8 works thought.
Here's the console error I get:
The text was updated successfully, but these errors were encountered: