Last modified: 2014-04-16 18:51:45 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T66005, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 64005 - Extra space before list item, Incompatible constraints (merge)
Extra space before list item, Incompatible constraints (merge)
Status: NEW
Product: Parsoid
Classification: Unclassified
serializer (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Gabriel Wicke
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-04-16 17:11 UTC by C. Scott Ananian
Modified: 2014-04-16 18:51 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description C. Scott Ananian 2014-04-16 17:11:19 UTC
This looks like it was caused by bug 50771.

$ echo "<ul><li><ul><li>x</li></ul>ul>" | tests/parse.js --html2wt
warning in enwiki:Main_Page: Incompatible constraints (merge): { a: { min: 0, max: 1 }, b: { min: 1, max: 2 }, min: 1, max: 1 } { a: { min: 0, max: 1 },
  b: {},
  min: 0,
  max: 0,
  constraintInfo: { onSOL: true, sepType: 'parent-child', nodeA: 'UL', nodeB: 'LI' } } { a: {}, b: { min: 1, max: 2 }, min: 1, max: 2 }
** x

Without data-parsoid hints, I guess the space insertion is inevitable -- but we shouldn't be causing incompatible constraints errors.
Comment 1 C. Scott Ananian 2014-04-16 17:57:42 UTC
Whoops, it should be:

$ echo "<ul><li><ul><li>x</li></ul></li></ul>" | tests/parse.js --html2wt

Weird cut-and-paste issue.  But the output is accurate.
Comment 2 C. Scott Ananian 2014-04-16 18:51:45 UTC
And note that the php parser generates:
<li> x
for * x
and
<li>x
for *x

So I don't the resolution of bug 50771 was correct. If VE wants a space before the list item by default, it should be generating a space as the first character inside the <li>.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links