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

Fixed issue #677 #868

Merged
merged 2 commits into from
May 30, 2013
Merged

Fixed issue #677 #868

merged 2 commits into from
May 30, 2013

Conversation

sarvaje
Copy link
Contributor

@sarvaje sarvaje commented May 13, 2013

Hello,

With this versions of windows, IE can't reproduce html5 media elements, but all functions are aviable, so if you check video.canPlayType !== 'undefined' return true, but if you call the function video.canPlayType("video/mp4") then a exception is throwed.

I add a piece of code that after check video.canPlayType !== 'undefined', I put a try/catch, to check if function work or not, if function throw exception, I set supportsMediaTag = false.

@sarvaje
Copy link
Contributor Author

sarvaje commented May 13, 2013

related with #677

@johndyer
Copy link
Collaborator

Thanks! Could you resubmit this without changes to the /build/ folder?

@sarvaje
Copy link
Contributor Author

sarvaje commented May 27, 2013

Sure,

Tomorrow or the day after tomorrow I resubmit without change on build folder

@sarvaje
Copy link
Contributor Author

sarvaje commented May 30, 2013

Hello,

@johndyer I restore the files on build folder to the previous versión.

@johndyer
Copy link
Collaborator

Sorry about that. We had a branch problem in merging. Will do.

On Thu, May 30, 2013 at 10:36 AM, David García notifications@github.comwrote:

Hello,

@johndyer https://github.com/johndyer I restore the files on build
folder to the previous versión.


Reply to this email directly or view it on GitHubhttps://github.com//pull/868#issuecomment-18667977
.

John Dyer - http://j.hn/

@sarvaje
Copy link
Contributor Author

sarvaje commented May 30, 2013

Can I do something?

johndyer added a commit that referenced this pull request May 30, 2013
@johndyer johndyer merged commit 4ef8d80 into mediaelement:master May 30, 2013
phuongdh pushed a commit to avalonmediasystem/mediaelement that referenced this pull request Oct 18, 2013
rexblack pushed a commit to rexblack/mediaelement that referenced this pull request May 22, 2014
rexblack pushed a commit to rexblack/mediaelement that referenced this pull request May 22, 2014
marmite22 pushed a commit to elucidat/mediaelement that referenced this pull request Dec 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants