Page MenuHomePhabricator

Update the diff table of contents to use hierarchical views and edit distance renames
ClosedPublic

Authored by epriestley on Apr 28 2020, 7:23 PM.
Tags
None
Referenced Files
F13084628: D21183.diff
Wed, Apr 24, 11:12 PM
Unknown Object (File)
Sun, Apr 21, 5:54 PM
Unknown Object (File)
Fri, Apr 19, 4:42 AM
Unknown Object (File)
Sun, Mar 31, 3:14 PM
Unknown Object (File)
Sun, Mar 31, 11:34 AM
Unknown Object (File)
Thu, Mar 28, 7:10 PM
Unknown Object (File)
Feb 16 2024, 2:57 AM
Unknown Object (File)
Feb 14 2024, 3:25 AM
Subscribers
None

Details

Summary

Ref T13520. Generally, make the table of contents look and more like the paths panel:

  • Show a hierarchy, with compression for single-sibling children.
  • Use the same icons, instead of "M D" and "(img)" stuff.
  • Use EditDistanceMatrix to do a piece-by-piece diff of paths changes.
  • Show path changes within the path list.

I'm not entirely sold on this, but it was complicated to write and I've never heard the term "sunk cost fallacy". I think this is mostly a net improvement, but may need some adjustments and followup.

Test Plan

Viewed various changes in Differential and Diffusion, saw a more usable table of contents.

Diff Detail

Repository
rP Phabricator
Branch
toc1
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/infrastructure/diff/view/PHUIDiffTableOfContentsListView.php:134XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 24264
Build 33420: Run Core Tests
Build 33419: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Apr 28 2020, 7:27 PM
This revision was automatically updated to reflect the committed changes.