Page MenuHomePhabricator

Use timeline view in Differential and make inlines somewhat usable again
ClosedPublic

Authored by epriestley on Feb 13 2014, 8:09 PM.
Tags
None
Referenced Files
F19890752: D8215.id19549.diff
Thu, Mar 26, 6:11 AM
F19855864: D8215.diff
Fri, Mar 13, 4:09 PM
F19831429: D8215.id19548.diff
Tue, Mar 10, 7:35 PM
F19769658: D8215.id19548.diff
Feb 19 2026, 8:22 PM
F19498366: D8215.id19548.diff
Jan 8 2026, 12:59 PM
F19152012: D8215.id19549.diff
Dec 11 2025, 12:10 AM
F19050707: D8215.id19549.diff
Nov 27 2025, 9:09 PM
F18878318: D8215.id.diff
Nov 6 2025, 4:53 PM
Subscribers

Details

Summary

Ref T2222. This gets rid of Differential's custom view and uses a standard view instead.

This also mostly fixes the rendering logic for inlines.

This is headed to the tmp.differential branch.

Test Plan

{F112696}

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/applications/differential/view/DifferentialTransactionView.php
19–21

I think I can put the grouping stuff in the core after we merge with a method like isThisKindOfTransactionAnInlineComment(), but the rendering I probably can't merge unless we change one of the applications.

epriestley updated this revision to Unknown Object (????).Feb 13 2014, 8:28 PM
  • Fix local anchor.
epriestley updated this revision to Unknown Object (????).Feb 13 2014, 8:50 PM

Fixes T1790.