I noticed that IE 10 was complaining that there was to many HTML tags on the page, and I believe it is right.

This are the conditional comments for in 2.8:

<!--[if IEMobile 7]><html class="no-js iem7" <?php print $html_attributes; ?>><![endif]-->
<!--[if (gt IEMobile 7)|!(IEMobile)]><!--><html class="no-js" lang="en"><!--<![endif]-->
<!--[if lt IE 7]><html class="no-js ie6 oldie" <?php print $html_attributes; ?>><![endif]-->
<!--[if (IE 7)&!(IEMobile)]><html class="no-js ie7 oldie" <?php print $html_attributes; ?>><![endif]-->
<!--[if (IE 8)&!(IEMobile)]><html class="no-js ie8 oldie" <?php print $html_attributes; ?>><![endif]-->
<!--[if gt IE 8]><!--><html class="no-js" <?php print $html_attributes; ?>><!--<![endif]-->
<!--[if (gte IE 9)|(gt IEMobile 7)]><!--><html class="no-js"  <?php print $rdf_namespaces; ?>><!--<![endif]-->

That results in IE9 and IE10 seeing three HTML tags, as far as I can see.

This is how I think they should be:

<!--[if IEMobile 7]><html class="no-js iem7"  lang="da" dir="ltr"><![endif]-->
<!--[if (gt IEMobile 7)|!(IEMobile)]><html class="no-js" lang="en"><![endif]-->
<!--[if lt IE 7]><html class="no-js ie6 oldie"  lang="da" dir="ltr"><![endif]-->
<!--[if (IE 7)&!(IEMobile)]><html class="no-js ie7 oldie"  lang="da" dir="ltr"><![endif]-->
<!--[if (IE 8)&!(IEMobile)]><html class="no-js ie8 oldie"  lang="da" dir="ltr"><![endif]-->
<!--[if gt IE 8]><html class="no-js"  lang="da" dir="ltr"><![endif]-->
<!--[if (gte IE 9)|(gt IEMobile 7)]><!--><html class="no-js" ><!--<![endif]-->

.

Comments

mortendk’s picture

Status: Active » Closed (fixed)

i cleaned it up and removed a lot of the options keeping it down to ie7 & ie8 in last version (7.x-2.9)