savepublishing icon indicating copy to clipboard operation
savepublishing copied to clipboard

Parse error: http://www.theverge.com/2013/1/9/3857542/hulu-we-reduced-the-number-of-ads-this-month-more-cuts-coming

Open hoffmanjeff opened this issue 13 years ago • 4 comments

hoffmanjeff avatar Jan 10 '13 17:01 hoffmanjeff

This and the other Verge parse errors are due to a SCRIPT tag that appears inside a SPAN. It considers a span "textish" and doesn't try to dig inside it further, so all this email address hiding js gets treated like text.

In general this is probably non-awesome, since people put things inside spans all the time.

kuro5hin avatar Jan 11 '13 20:01 kuro5hin

Considering span a block element fixes this.

kuro5hin avatar Jan 11 '13 21:01 kuro5hin

The way span is used for NAV and for inline textual stuff might blow up the page.

ftrain avatar Jan 11 '13 21:01 ftrain

I haven't done very much testing but it doesn't seem to. I guess I'd make the argument that people use span wrong in all sorts of creative ways, which they tend not to with the other elements were considering textish.

kuro5hin avatar Jan 11 '13 21:01 kuro5hin