Page MenuHomePhabricator

Allow CustomField to provide ApplicationTransaction change details
ClosedPublic

Authored by epriestley on Feb 20 2014, 6:41 PM.
Tags
None
Referenced Files
F19827531: D8284.diff
Mon, Mar 9, 10:01 AM
F19730203: D8284.id.diff
Sat, Feb 14, 1:30 PM
F19601501: D8284.id19725.diff
Feb 3 2026, 4:56 AM
F19575760: D8284.id19725.diff
Feb 1 2026, 7:52 AM
F19509893: D8284.id.diff
Jan 11 2026, 2:30 AM
F19106408: D8284.diff
Dec 7 2025, 4:16 AM
F18875157: D8284.diff
Nov 5 2025, 4:58 PM
F18842528: D8284.diff
Oct 28 2025, 4:17 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.