ml-tsai / curved-corner

Automatically exported from code.google.com/p/curved-corner
0 stars 0 forks source link

Check for IE9 #78

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Please add next check into oncountready() function, to avoid using 
curved-corner in IE9:

if (!document.documentMode || document.documentMode > 8) { return; }

Original issue reported on code.google.com by Maxim.Gura@gmail.com on 9 Sep 2011 at 12:53

GoogleCodeExporter commented 8 years ago
Considering the behavior property is not compliant with the CSS standards -- to 
produce a valid document, the stylesheet with the behavior would need to be 
enclosed within an IE conditional comment, e.g.:

<!--[if lte IE 8]>
<style type="text/css">
.curved{ behavior:url( "border-radius.htc" ); }
</style>
<![endif]-->

IE9 supports border-radius natively, so there's no reason to include the 
behavior on an IE9 version of a page.  If conditional comments are used, then 
the if statement isn't necessary because the behavior would not be loaded in 
IE.  In short, this is not a defect -- the script must only target versions of 
IE that do not support border-radius.  The demo doesn't currently use IE 
conditional comments to load the behavior -- these will be updated in the near 
future.

Original comment by tyler.wa...@gmail.com on 2 Oct 2011 at 10:07