Implement UI changes for diff view to make moved paragraphs more visible.
Description
Description
| Status | Assigned | Task | ||
|---|---|---|---|---|
| Resolved | DannyH | T121469 Improve diff compare screen | ||
| Open | None | T139603 Show text changes when moving text chunks (#2) | ||
| Open | jkroll | T146781 Show changes in moved text chunks (C++) | ||
| Open | jkroll | T160077 UI changes for showing changes in moved text chunks | ||
| Open | None | T140340 Connect moved paragraphs by arrows | ||
| Open | jkroll | T166882 Add tooltip to unicode arrow indicating a moved paragraph |
Comment Actions
Taking Tim Starling's feedback on Gerrit into account, I've inserted named anchors in the diff output for moved paragraphs. The + and - symbols are replaced with "left shift" (<<) and "right shift" (>>) symbols. Hyperlinks allow to move between source and destination paragraphs. This is a good solution because
- it shows which paragraphs were moved, even if they weren't changed
- it's plain text, so it can be searched for in the browser
- it doesn't require JavaScript or visual changes which have to be adjusted to each Skin, such as different background colors for moved blocks
The code is straightforward and doesn't have performance implications, so it doesn't require a separate patch set.
Example links:
https://wmde-wikidiff2-patched.wmflabs.org/core/index.php?title=Red_imported_fire_ant&diff=24&oldid=23
https://wmde-wikidiff2-patched.wmflabs.org/core/index.php?title=Red_imported_fire_ant&diff=25&oldid=24
https://wmde-wikidiff2-patched.wmflabs.org/core/index.php?title=Red_imported_fire_ant&diff=26&oldid=25
Feedback welcome.