25572 2014-05-06 06:57:11 +0000 Don't remove script elements when checking noscript (hgroup content model) 2014-07-29 23:50:20 +0000 1 1 1 Unclassified WHATWG HTML unspecified Other other RESOLVED FIXED http://www.whatwg.org/specs/web-apps/current-work/#the-noscript-element P3 normal Unsorted 1 contributor ian hsivonen ian mike zcorpan contributor oldest_to_newest 105161 0 contributor 2014-05-06 06:57:11 +0000 Specification: http://www.whatwg.org/specs/web-apps/current-work/multipage/scripting-1.html Multipage: http://www.whatwg.org/C#the-noscript-element Complete: http://www.whatwg.org/c#the-noscript-element Referrer: http://www.whatwg.org/specs/web-apps/current-work/multipage/ Comment: Don't remove script elements when checking noscript What is the point of removing the script elements? It seems to me it affects other parts of the document, e.g. <hgroup><script></script></hgroup> <p><noscript>hello</noscript></p> Removing the script makes the hgroup be invalid. Posted from: 2a00:801:e0:30:280f:e9ec:404a:b94e User agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/34.0.1847.116 Safari/537.36 OPR/21.0.1432.48 (Edition Next) 105213 1 ian 2014-05-06 18:06:34 +0000 Hm, yeah, <hgroup> with just a <script> probably shouldn't be valid. It would break in non-script scenarios. That's a good argument for removing it from <hgroup>. Is there anything else this breaks? 105293 2 zcorpan 2014-05-07 10:56:55 +0000 I think hgroup is the only odd one. 105324 3 ian 2014-05-07 18:11:05 +0000 Let's just drop <script> from <hgroup> then. 109507 4 ian 2014-07-29 17:08:51 +0000 Ok, this is done. I just changed h1-h6,script,template to h1-h6,template as the allowed children of <hgroup>. The checkin is going to be delayed because I'm still trying to pin down a very subtle bug in my pipeline.