Permalink
Browse files

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 0ea342a

Refs gh-2031
Refs gh-2002
Fixes gh-2493
Closes gh-2499
  • Loading branch information...
anthonyryan1 authored and mgol committed Jul 25, 2015
1 parent b5f7c9e commit 99e8ff1baa7ae341e94bb89c3e84570c7c3ad9ea
Showing with 6 additions and 13 deletions.
  1. +6 −13 src/manipulation/wrapMap.js
@@ -6,20 +6,13 @@ var wrapMap = {
// Support: IE9
option: [ 1, "<select multiple='multiple'>", "</select>" ],
// XHTML parsers do not magically insert elements in the
// same way that tag soup parsers do. So we cannot shorten
// this by omitting <tbody> or other required elements.
thead: [ 1, "<table>", "</table>" ],
// 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, "<table>", "</table>" ],
// Auto-insert "tbody" element
tr: [ 2, "<table>", "</table>" ],
// Auto-insert "tbody" and "tr" elements
td: [ 3, "<table>", "</table>" ],
col: [ 2, "<table><colgroup>", "</colgroup></table>" ],
tr: [ 2, "<table><tbody>", "</tbody></table>" ],
td: [ 3, "<table><tbody><tr>", "</tr></tbody></table>" ],
_default: [ 0, "", "" ]
};

0 comments on commit 99e8ff1

Please sign in to comment.