Page MenuHomePhabricator

Visual diffing: indenting a list item produces an incorrect diff
Closed, ResolvedPublic8 Estimated Story Points

Description

Indent the item 'or here' on Simple.html and an exception is thrown when diffing. incorrect diff is produced:

pasted_file (134×192 px, 3 KB)

Event Timeline

Jdforrester-WMF set the point value for this task to 1.
Jdforrester-WMF added a subscriber: Tchanders.

Change 319416 had a related patch set uploaded (by Tchanders):
Fix splice typo

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

Esanders renamed this task from Visual diffing: indenting a list item throws an exception to Visual diffing: indenting a list item produces an incorrect diff.Nov 2 2016, 10:05 PM
Esanders updated the task description. (Show Details)

The above patch fixes the exception (which was the same as T149528) so re-purposing this for a follow-up issue.

Actually the patch improved the rendering, but it still shows one item removed and two inserted:

pasted_file (243×456 px, 26 KB)

Esanders changed the point value for this task from 1 to 8.Jan 17 2017, 5:21 PM

Change 340656 had a related patch set uploaded (by Tchanders):
[VisualEditor/VisualEditor] WIP Clean up DiffElement logic

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

Change 340656 merged by jenkins-bot:
[VisualEditor/VisualEditor] Generalize DiffElement logic (improves lists and tables)

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

Change 342651 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (bc6417ba5)

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

Change 342651 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (41134af2b)

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