On 08/20/2013 01:21 PM, Emmanuel Engelhart wrote:

Hi,

Have a look to this rendering: http://parsoid.wmflabs.org/fi/Kypros

The parser adds a bunch of this line at the beginning: </span><p data-parsoid='{"fostered":true,"autoInsertedEnd":true}' about="#mwt2"><br data-parsoid="{}"></p><span about="#mwt2" data-parsoid="{}">

Looks pretty strange and, to me, not understandable... Bug or Feature?

The fostered content detection is a feature. It marks content that was fostered out of a table (see http://www.whatwg.org/specs/web-apps/current-work/multipage/tree-constructio...), so that we don't create a dirty diff when round-tripping this content.

The <p><br></p> that was fostered here hints at newlines in the template expansion, which then trigger the p/br insertion. Those might not be correct, as the PHP parser + tidy does not seem to emit them. I'll have a look.

Gabriel

Le 24/08/2013 01:20, Gabriel Wicke a écrit :

...

The <p><br></p> that was fostered here hints at newlines in the template expansion, which then trigger the p/br insertion. Those might not be correct, as the PHP parser + tidy does not seem to emit them. I'll have a look.

This was indeed not correct. Unbalanced table tags threw our 'in table' state tracking on the token stream off. Reported in https://bugzilla.wikimedia.org/show_bug.cgi?id=53287, and should be fixed by https://gerrit.wikimedia.org/r/#/c/80681/.

Great. Thank you. Emmanuel