Page MenuHomePhabricator

Allow CustomField to provide ApplicationTransaction change details
ClosedPublic

Authored by epriestley on Feb 20 2014, 6:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 18, 2:38 PM
Unknown Object (File)
Mon, Feb 17, 1:27 PM
Unknown Object (File)
Mon, Feb 17, 1:27 PM
Unknown Object (File)
Sun, Feb 9, 11:35 AM
Unknown Object (File)
Sun, Feb 9, 11:34 AM
Unknown Object (File)
Sun, Feb 9, 11:34 AM
Unknown Object (File)
Sat, Feb 8, 7:33 PM
Unknown Object (File)
Fri, Jan 31, 11:44 PM
Subscribers

Details

Summary

Ref T3886. Ref T418. For fields like "Summary" and "Test Plan" where changes can't be summarized in one line, allow CustomField to provide a "(Show Details)" link and render a diff.

Also consolidate some of the existing copy/paste, and simplify this featuer slightly now that we've move to dialogs.

Test Plan

{F115918}

  • Viewed "description"-style field changes in phlux, pholio, legalpad, maniphest, differential, ponder (questions), ponder (answers), and repositories.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Dialog seems like a better UI to me. Curious to see if we get any reactions.