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)
Fri, Nov 8, 7:20 PM
Unknown Object (File)
Oct 24 2024, 5:56 AM
Unknown Object (File)
Oct 17 2024, 12:26 PM
Unknown Object (File)
Oct 16 2024, 11:11 AM
Unknown Object (File)
Oct 9 2024, 9:48 PM
Unknown Object (File)
Oct 9 2024, 6:49 PM
Unknown Object (File)
Oct 9 2024, 5:18 PM
Unknown Object (File)
Oct 9 2024, 5:15 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.