Indent the item 'or here' on Simple.html and an exception is thrown when diffing. incorrect diff is produced:
Description
Description
Details
Details
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Open | None | T143341 When re-starting a local draft, show the user a diff so they know what they're getting into | |||
Open | None | T105173 HTML diffs of edits for everything | |||
Resolved | Tchanders | T143350 Let users choose a visual or wikitext diff in the review-your-changes tab | |||
Resolved | Tchanders | T149537 Visual diffing: indenting a list item produces an incorrect diff |
Event Timeline
Comment Actions
The above patch fixes the exception (which was the same as T149528) so re-purposing this for a follow-up issue.
Comment Actions
Actually the patch improved the rendering, but it still shows one item removed and two inserted:
Comment Actions
Change 340656 had a related patch set uploaded (by Tchanders):
[VisualEditor/VisualEditor] WIP Clean up DiffElement logic
Comment Actions
Change 340656 merged by jenkins-bot:
[VisualEditor/VisualEditor] Generalize DiffElement logic (improves lists and tables)
Comment Actions
Change 342651 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (bc6417ba5)
Comment Actions
Change 342651 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (41134af2b)