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)
Wed, Apr 17, 2:09 AM
Unknown Object (File)
Sat, Apr 6, 11:14 PM
Unknown Object (File)
Sat, Apr 6, 1:17 AM
Unknown Object (File)
Thu, Apr 4, 7:31 PM
Unknown Object (File)
Thu, Apr 4, 11:17 AM
Unknown Object (File)
Sun, Mar 31, 7:44 PM
Unknown Object (File)
Sat, Mar 30, 9:45 AM
Unknown Object (File)
Mar 20 2024, 12:48 AM
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

Repository
rP Phabricator
Branch
dfield2
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

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