CDATA section in SVG title is handled differently in browsers

[zcorpan]

Test case http://software.hixie.ch/utilities/js/live-dom-viewer/saved/6207

Edge and Firefox support the CDATA section. Safari and Chrome insert a comment.

I think Edge and Firefox match the spec,

https://html.spec.whatwg.org/multipage/parsing.html#markup-declaration-open-state

"If there is an adjusted current node and it is not an element in the HTML namespace, then switch to the CDATA section state."

FYI @whatwg/html-parser

I think the spec can probably stay as-is, but we should make sure there is a test and file bugs.

[gsnedders]

I'd agree the spec is right here.

[zcorpan]

Hmm it seems like CDATA sections are just not at all supported in Safari and Chrome.

data:text/html,<svg><!|CDATA[ ><p>FAIL

Edit: made a typo.

[zcorpan]

https://bugs.chromium.org/p/chromium/issues/detail?id=881983
https://bugs.webkit.org/show_bug.cgi?id=189431

[zcorpan]

https://github.com/html5lib/html5lib-tests/blob/515dc09aaa25d48a72f0d4c1a890507022b36e09/tree-construction/tests21.dat has tests for CDATA in svg/math, could add more tests to make sure it's supported in SVG title, but not in HTML span in SVG title.

[scshepard]

Not a browser expert here, but chrome and opera show: * #comment: [CDATA[x]] Edge shows: #text: x I don't have mac access for safari Blisk: * #comment: [CDATA[x]] Steve

…

________________________________ From: Simon Pieters <notifications@github.com> Sent: Friday, September 7, 2018 8:04:48 PM To: whatwg/html Cc: Subscribed Subject: [whatwg/html] CDATA section in SVG title is handled differently in browsers (#4016) Test case http://software.hixie.ch/utilities/js/live-dom-viewer/saved/6207 Edge and Firefox support the CDATA section. Safari and Chrome insert a comment. I think Edge and Firefox match the spec, https://html.spec.whatwg.org/multipage/parsing.html#markup-declaration-open-state "If there is an adjusted current node and it is not an element in the HTML namespace, then switch to the CDATA section state." FYI @whatwg/html-parser<https://github.com/orgs/whatwg/teams/html-parser> I think the spec can probably stay as-is, but we should make sure there is a test and file bugs. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#4016>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ACKXS5Phk0Ie32fkS08SIf4hyz1nAtYfks5uYtFggaJpZM4WfZy->.