Page MenuHomePhabricator

Image inside a table incorrectly removes a </span> tag
Closed, ResolvedPublic

Details

Reference
bz50976

Event Timeline

bzimport raised the priority of this task from to High.Nov 22 2014, 2:00 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz50976.

Can we get a simpler explanation of the bug? It's hard for me to see exactly what's going on here.

I think the original issue has been fixed (?), but note the insertion of ~ 10,000px worth of height of empty HTML in the infobox table:

<tr data-parsoid="{&quot;startTagSrc&quot;:&quot;|-&quot;}">
<td data-parsoid="{&quot;autoInsertedStart&quot;:true,&quot;autoInsertedEnd&quot;:true}">

<p data-parsoid="{}"><br data-parsoid="{}"></p>

[…]

<p data-parsoid="{}"><br data-parsoid="{}"></p></td></tr>

– see http://parsoid-lb.eqiad.wikimedia.org/ptwiki/Scapanulus_oweni?oldid=34667801

I suspect that the empty HTML is in fact a symptom of the fact that the PHP parser omits 'empty' table rows, while parsoid doesn't. nell-wikipedia had a CSS row to suppress the empty rows.

...although it seems that the rows in this case aren't, strictly speaking, empty? This merits some investigation.

This was fixed as part of T69452 where empty rows emitted by templates are stripped by parsoid as well.