Page MenuHomePhabricator

Enforce single-line context in the serializer
Closed, ResolvedPublic0 Estimated Story Points

Description

We need to enforce single-line syntactical context in headings and other line-based constructs (lists, paragraphs, etc pp).

Simple test case:

echo -e '<h2>\nfoo</h2>' | node parse --html2wt
==
foo==

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 22 2014, 2:09 AM
bzimport set Reference to bz50683.

See also bug 51444. I'm not merging these bugs because 51444 is filed against VE, and I'm not sure that VE is blameless here (for including the newline in the heading to begin with).

Gwicke's initial commentary on this issue is at http://pastie.org/8146028

Arlolra added a project: Parsoid.
Arlolra set Security to None.
Arlolra removed a subscriber: MarkTraceur.

Change 196246 had a related patch set uploaded (by Arlolra):
Enforce single-line context in the serializer

https://gerrit.wikimedia.org/r/196246

Change 196246 merged by jenkins-bot:
Enforce single-line context in the serializer

https://gerrit.wikimedia.org/r/196246