[fix]: IE® conditional comments

Question

[fix]: IE® conditional comments

stevenvachon opened this issue 8 years ago · comments

Downlevel-hidden, -revealed and bubbling, so that we can correctly parse this:

<html>
<!--[if lt IE 7]><html class="no-js ie6"><![endif]-->
<!--[if IE 7]><html class="no-js ie7"><![endif]-->
<!--[if IE 8]><html class="no-js ie8"><![endif]-->
<!--[if gt IE 8]><!--><html class="no-js"><!--<![endif]-->

(Not that I condone writing such filth)

Ivan Demidov · Answer 1 · Fri Dec 15 2017 18:06:09 GMT+0800 (China Standard Time)

Hi @stevenvachon this related to:

Add support for condtional comment

I hope that you no longer use such constructions

Steven Vachon · Answer 2 · Fri Dec 15 2017 23:45:54 GMT+0800 (China Standard Time)

@gitscrum cnn.com does

Sukka · Answer 3 · Mon Nov 23 2020 20:28:33 GMT+0800 (China Standard Time)

Although it is possible to extract the content of the conditional comment and pass it back to postHtmlParser() again (just as I did in posthtml/htmlnano#119), however, the code snippet provided only has <html> start tag wrapped in a conditional comment, posthtml-render later will have to determine if </html> should be added. In posthtml/htmlnano#125 I have to manually remove </html> from result.html (kind of a dirty hack).

Ivan Demidov · Answer 4 · Mon Nov 23 2020 20:34:36 GMT+0800 (China Standard Time)

I thought the same, but it's very dirty

Ivan Demidov · Answer 5 · Mon Aug 08 2022 18:22:59 GMT+0800 (China Standard Time)

I think that it makes no sense to invest a resource in supporting something that is no longer supported.

Feel free to reopen the issue if you think it's a high priority and a must for you.