Page MenuHomePhabricator

Difference in dd to dt transition
Closed, ResolvedPublic

Description

https://prv-tests.wmcloud.org/diff/ruwiki/%D0%A8%D0%B0%D0%B1%D0%BB%D0%BE%D0%BD:%D0%A1%D0%BE%D0%B2%D0%B5%D1%82%20%D0%A4%D0%B5%D0%B4%D0%B5%D1%80%D0%B0%D1%86%D0%B8%D0%B8

$ cat t
; test
: 123
:: 456
; odd
: asdf

$ cat t | php maintenance/run parse
<dl><dt>test</dt>
<dd>123
<dl><dd>456</dd></dl></dd></dl>
<dl><dt>odd</dt>
<dd>asdf</dd></dl>

$ cat t | php maintenance/run parse --parsoid
<section data-mw-section-id="0" id="mwAQ"><dl id="mwAg"><dt id="mwAw">test</dt>
<dd id="mwBA">123
<dl id="mwBQ"><dd id="mwBg">456</dd></dl></dd>
<dt id="mwBw">odd</dt>
<dd id="mwCA">asdf</dd></dl>
</section>

Event Timeline

Parsoid's handling seems reasonable to me. Should this be declined as a known difference?

Parsoid's behavior is not just reasonable but preferable. (See also T223277: List embedded in description list <dd> (descriptions) causes LISTGAP issues (old Parser only; does not affect Parsoid) which I think is the same bug identified for Parser.php or close enough to it.)

ABreault-WMF claimed this task.
ABreault-WMF added a project: Parsoid.
ABreault-WMF moved this task from Needs Triage to Known Differences on the Parsoid board.