Page MenuHomePhabricator

Images: Invalid image parameter gets removed, but so does a valid one next to it
Closed, ResolvedPublic

Description

https://it.wikipedia.org/w/index.php?title=Terza_guerra_di_indipendenza_italiana&curid=232854&diff=64604490&oldid=64391386
In this edit "float|left" is removed, which is not a problem for the first one, but apparently it is for the second one.


Version: unspecified
Severity: normal

Details

Reference
bz62500

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 2:58 AM
bzimport added a project: Parsoid.
bzimport set Reference to bz62500.

Related:
https://it.wikipedia.org/w/index.php?title=Casco&diff=next&oldid=64464748
removes "righ" . This is arguably a selser bug in VE (bug 62499) but it might be possible for us to preserve invalid image options better in parsoid.

Change 118014 had a related patch set uploaded by Cscott:
WIP: fix parsing of invalid image options.

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

Change 118216 had a related patch set uploaded by Cscott:
Remove img_float.

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

Change 118014 merged by jenkins-bot:
Parse and round-trip even invalid image options.

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