-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
No effect in IE8 #1054
Comments
Yes, there are multiple IE8 error reports, and I'm checking if it's a VM issue or a real issue with IE8 since I can't replicate it on my VM |
see also #1061. there you can see combinations where we have problems. let me know if i can test something for you. |
any update? |
no |
Were either of you guys able to find a previous version of MediaElement.js that worked with your particular IE8/Flash/VM combination?
|
I had a similar issue and what I noticed was that the "embed" tag was being used instead of the "object" to embed the flash file. Then I looked at the IE detection code and it seems that IE is not being detected correctly. t.isIE = (nav.appName.toLowerCase().match(/trident/gi) !== null); "trident" is usually found in the user-agent string, not the appName from what I understand. So I changed the above line of code to: t.isIE = (ua.match(/trident/gi) !== null); And the video and video controls started working in IE8 just fine. |
@johndyer there is a comment from 5months ago from @jefferyto what could help. |
@FlashJunior I opened a pull request (#1018) that was merged about 2 months ago. Perhaps you can try testing the latest version from master (if you haven't already)? |
@jefferyto but if you look @ https://github.com/johndyer/mediaelement/blob/master/build/mediaelement-and-player.js in the master you see just |
i build a test-page. 2.13.1: is not working with IE8 and older flashplayer then 11.9
2.13.0: is working with IE8 and older flashplayer then 11.9
2.12.0: is working with IE8 and older flashplayer then 11.9
compared: 2.13.0...2.13.1 Screenshot and more: #1040 (comment) |
@FlashJunior The files in |
ah, i see. in https://github.com/johndyer/mediaelement/blob/master/src/js/me-featuredetection.js is en other detection. so @johndyer should update the build folder :-/ |
Yes. I only update the build folder when a release is done. I haven't been able to do this until the IE8 issue is 100% resolved. JD
|
@johndyer if you like i can test for you with an old win XP IE8 where i get the errors on 2.13.1 version. Send me the compiled version to fj@flashjunior.ch |
@FlashJunior Just checking here as well. My IE8 works with video on http://www.onebyte.ch/preview/mediaelementjs/v2_13_0/demo/mediaelementplayer.html but I'm still getting a strange error on the audio. I think it might be my Flash install or the fact that I'm using Flash 12.0 |
Could you try to updated 2.13.2? |
i updated http://www.onebyte.ch/preview/mediaelementjs/ |
I thought it was my problem, but I use your demos ... still the same does not work, I hope to be resolved
The text was updated successfully, but these errors were encountered: