Closed (fixed)
Project:
mothership
Version:
7.x-2.8
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
12 Oct 2012 at 15:18 UTC
Updated:
2 Jul 2013 at 12:56 UTC
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
Comment #1
mortendk commentedi cleaned it up and removed a lot of the options keeping it down to ie7 & ie8 in last version (7.x-2.9)