Paragraph wrapper introduces <p></p> in output HTML -- investigate and kill them where they are a result of edge case diffs between PHP parser and Parsoid code
Open, NormalPublic

Description

See discussion @ https://gerrit.wikimedia.org/r/c/mediawiki/services/parsoid/+/476053/3/tests/parserTests.txt#31161

Most of the empty p-tags are likely edge case bugs.

ssastry created this task.Wed, Nov 28, 7:27 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptWed, Nov 28, 7:27 PM
ssastry triaged this task as Normal priority.Wed, Nov 28, 7:27 PM
ssastry renamed this task from Paragraph wrapper introduces <p></p> in o utput HTML -- investigate and kill them where they are a result of edge case diffs between PHP parser and Parsoid code to Paragraph wrapper introduces <p></p> in output HTML -- investigate and kill them where they are a result of edge case diffs between PHP parser and Parsoid code.

This test "Empty paragraphs (marked with mw-empty-elt) found in source should not be normalized away" should be removed after this is fixed.

Change 476352 had a related patch set uploaded (by Arlolra; owner: Arlolra):
[mediawiki/services/parsoid@master] Stop matching non-breaking spaces in TokenUtils.isSolTransparent

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

Arlolra claimed this task.Wed, Nov 28, 7:54 PM

Change 476352 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Stop matching non-breaking spaces in TokenUtils.isSolTransparent

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

Change 477686 had a related patch set uploaded (by Arlolra; owner: Arlolra):
[mediawiki/services/parsoid@master] Don't migrate out trailing non-breaking spaces

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

Change 477686 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Don't migrate out trailing non-breaking spaces

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

Mentioned in SAL (#wikimedia-operations) [2018-12-05T21:39:08Z] <arlolra> Updated Parsoid to a6058e3 (T210647, T208360, T205333)