Page MenuHomePhabricator

In prose diff dialogs (like "Show Details" in transactions), show "old", "new" and "diff" tabs
ClosedPublic

Authored by epriestley on Nov 7 2016, 7:23 PM.
Tags
None
Referenced Files
F14307799: D16817.diff
Tue, Dec 17, 11:38 PM
Unknown Object (File)
Sun, Dec 15, 11:07 PM
Unknown Object (File)
Sat, Dec 7, 4:18 PM
Unknown Object (File)
Thu, Dec 5, 7:46 AM
Unknown Object (File)
Wed, Dec 4, 7:56 PM
Unknown Object (File)
Tue, Dec 3, 7:59 PM
Unknown Object (File)
Mon, Nov 25, 3:02 PM
Unknown Object (File)
Thu, Nov 21, 9:44 AM
Subscribers
None
Tokens
"Yellow Medal" token, awarded by avivey.

Details

Summary

Ref T7643. When you do something like this:

  • Edit a task description.
  • Click "Show Details" on the resulting transaction.
  • Get a prose diff dialog showing the change.

...now add some "Old" and "New" tabs. These are useful for:

  • reverting to the old text by copy/pasting;
  • reading just the new/old text if the diff is noisy;
  • sometimes just nice to have?

(This looks a little rough but I didn't want to put a negative margin on tab groups inside dialogs? Not sure what the best fix here is.)

Test Plan

Screen Shot 2016-11-07 at 11.18.39 AM.png (812×1 px, 88 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to In prose diff dialogs (like "Show Details" in transactions), show "old", "new" and "diff" tabs.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

You can add a class to the dialog, we do this with object-lists:

pasted_file (62×207 px, 8 KB)

chad edited edge metadata.
This revision is now accepted and ready to land.Nov 7 2016, 8:53 PM
epriestley edited edge metadata.
  • Uh just kind of fake it, maybe I'll clean this up when I fix Paste/Config code diffs? 🐗
This revision was automatically updated to reflect the committed changes.