10314 2010-08-08 04:02:29 +0000 "in cell" (secondary) insertion mode shows strange behavior for e.g. <table><tr><td><svg><td></td><td>, <table><tr><td><svg><tbody></tbody> 2010-10-04 14:49:28 +0000 1 1 1 Unclassified HTML WG pre-LC1 HTML5 spec (editor: Ian Hickson) unspecified Other other RESOLVED FIXED http://www.whatwg.org/specs/web-apps/current-work/#parsing-main-intd P3 normal --- 1 contributor ian eric hsivonen ian james mike public-html-admin public-html-wg-issue-tracking w3c public-html-bugzilla oldest_to_newest 37281 0 contributor 2010-08-08 04:02:29 +0000 Section: http://www.whatwg.org/specs/web-apps/current-work/#parsing-main-intd Comment: "in cell" (secondary) insertion mode shows strange behavior for e.g. <table><tr><td><svg><td></td><td>, <table><tr><td><svg><tbody></tbody> Posted from: 121.102.72.35 38842 1 ian 2010-09-10 19:20:08 +0000 I assume you mean <table><tr><td><svg><desc><td>? I guess the solution is to make it clear that "reprocess the current token" means that we should first check the rule in the "in foreign content" section that resets the insertion mode, before then rerunning the token through the entire system from the top? There is also the problem with <table><tr><td><svg><td><desc><td>. Not sure how to handle that one, unless we start checking for namespaces when looking at tag names. Any suggestions? We could probably solve both problems by making <svg> scoping for table scope, but then we'd have to add a bunch of error-handling logic in the table modes. Or maybe the real problem is that the secondary mode can be anything but "in body". Should we give up with the secondary mode idea, and just reset the insertion mode when exiting foreign lands, and always defer to the in-body mode from foreign lands when appropriate? 39559 2 ian 2010-09-27 09:13:07 +0000 Cases that would not be solved by the proposal above: <foo><svg></foo> <svg><em><desc></em> Maybe these aren't problems? It's a bigger issue if the implied end tags cause such problems... are there any that could trigger something like this if we always defer to "in body"? 39560 3 ian 2010-09-27 09:17:04 +0000 <svg><body><desc></html> <p><svg><desc><form> Maybe we should always defer to "in body" _and_ make the foreign lands scoping. 39562 4 ian 2010-09-27 09:19:16 +0000 Hm, <foreignObject> is already scoping. Maybe I should make all the other exits scoping as well, rather than make the entrances scoping. 39631 5 ian 2010-09-27 23:45:07 +0000 EDITOR'S RESPONSE: This is an Editor's Response to your comment. If you are satisfied with this response, please change the state of this bug to CLOSED. If you have additional information and would like the editor to reconsider, please reopen this bug. If you would like to escalate the issue to the full HTML Working Group, please add the TrackerRequest keyword to this bug, and suggest title and text for the tracker issue; or you may create a tracker issue yourself, if you are able to do so. For more details, see this document: http://dev.w3.org/html5/decision-policy/decision-policy.html Status: Accepted Change Description: see diff given below Rationale: Ok, I've tried to address this. This is a pretty big change in how foreign lands are built, it gets rid of the secondary insertion mode, etc. See the check-in comment below. 39632 6 contributor 2010-09-27 23:45:25 +0000 Checked in as WHATWG revision r5521. Check-in comment: Parser foreign lands: Get rid of secondary insertion mode, always use 'in body', scope HTML in foreign lands so that we won't try to close elements cross-namespace (e.g. <svg><p><desc><form> won't imply a </p> that closes the <p>...; <td> won't close all the SVG if the SVG is in a cell). http://html5.org/tools/web-apps-tracker?from=5520&to=5521 39633 7 contributor 2010-09-27 23:54:24 +0000 Checked in as WHATWG revision r5522. Check-in comment: Parser foreign lands: Oops, I missed one of the ways that scoping happens. http://html5.org/tools/web-apps-tracker?from=5521&to=5522 39634 8 eric 2010-09-27 23:57:32 +0000 https://bugs.webkit.org/show_bug.cgi?id=46676 is the WebKit bug for applying these spec changes. Thanks you Hixie.