-
Notifications
You must be signed in to change notification settings - Fork 3k
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 Running on Windows Server SKU can cause an exception to be thrown #224
Comments
Eessh. Can you file a ticket on IE's Connect website and report back here with the link? Thanks! |
The Connect site for IE9 was closed after it RTM'd. I did however contact a PM on the IE9 team (I work on the ASP.NET MVC team) and I will fill you in more when I hear back from him. For now modernizr could wrap those calls in a try catch because if anything errors out I would expect it to just say no it isn't supported. |
Hey Paul, This is a known issue for IE9. It sounds like that the best bet for now would be for there to be a try catch block around this call and treat a exception as false. |
lol. that is so effing weak. aiight thanks for digging into that osbornm. I'll either document the problem or try/catch it. |
I suggest both: try/catch it in the code and document it briefly in a comment around it, and then we'll include more detailed info in the full documentation. What a pain, IE9. |
Awesome thanks for getting this fix in. When is the time frame for the release of Modernizr 2? Do you have a date yet? |
Right now we're trying to get it out for Monday... :) so if not then, then quite soon |
Looks like it works. Thanks. |
…missing Desktop Experience feature or quicktime respectively. More info #224 * Wrapped the test in a try catch like it is in the video test.
…ase without "desktop experience" installed, means missing HTML5 audio/video support. Hat tip: Modernizr/Modernizr#224
… in some rare cases. details: it seems that if you don't have the "Desktop Experience" installed, using elem.canPlayType() will throw an exception. Awesome! So we try/catch it. It's okay, they're cheap. I'm doing a little bit of magic with an assignment inside my if() conditional. I hope nobody minds, too much. Fixes Modernizr#224
…missing Desktop Experience feature or quicktime respectively. More info Modernizr#224 * Wrapped the test in a try catch like it is in the video test.
If you are running IE9 on a windows server SKU without the "desktop experience" installed the "CanPlayType" method will throw a not implemented exception that will cause the modernizr script to fail.
Here is a simple repro that can be used on a machine running a default install of Windows Server 2008 R2 with IE9 RTM installed:
The text was updated successfully, but these errors were encountered: