From 99e8ff1baa7ae341e94bb89c3e84570c7c3ad9ea Mon Sep 17 00:00:00 2001 From: Anthony Ryan Date: Sat, 25 Jul 2015 02:15:02 -0400 Subject: [PATCH] Manipulation: Don't provide the parser with sloppy table markup While we can reply on parsers that were designed to cope with malformed syntax to understand what we mean, we shouldn't intentionally provide bad markup, not all parsers will accept it. "Be conservative in what you do, be liberal in what you accept from others." Reverts 0ea342a6a6dce793c1b0f14f051c2573f40f4e44 Refs gh-2031 Refs gh-2002 Fixes gh-2493 Closes gh-2499 --- src/manipulation/wrapMap.js | 19 ++++++------------- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/src/manipulation/wrapMap.js b/src/manipulation/wrapMap.js index a7b913c90..fdb430a03 100644 --- a/src/manipulation/wrapMap.js +++ b/src/manipulation/wrapMap.js @@ -6,20 +6,13 @@ var wrapMap = { // Support: IE9 option: [ 1, "" ], + // XHTML parsers do not magically insert elements in the + // same way that tag soup parsers do. So we cannot shorten + // this by omitting or other required elements. thead: [ 1, "", "
" ], - - // Some of the following wrappers are not fully defined, because - // their parent elements (except for "table" element) could be omitted - // since browser parsers are smart enough to auto-insert them - - // Auto-insert "colgroup" element - col: [ 2, "", "
" ], - - // Auto-insert "tbody" element - tr: [ 2, "", "
" ], - - // Auto-insert "tbody" and "tr" elements - td: [ 3, "", "
" ], + col: [ 2, "", "
" ], + tr: [ 2, "", "
" ], + td: [ 3, "", "
" ], _default: [ 0, "", "" ] };