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
F13224300: D16817.id.diff
Sun, May 19, 7:38 AM
F13222617: D16817.diff
Sun, May 19, 3:49 AM
F13218668: D16817.id40503.diff
Sat, May 18, 1:05 PM
F13215302: D16817.id40504.diff
Fri, May 17, 4:23 PM
F13205286: D16817.diff
Wed, May 15, 1:47 AM
F13189415: D16817.diff
Sat, May 11, 5:55 AM
Unknown Object (File)
Wed, May 1, 12:29 AM
Unknown Object (File)
Fri, Apr 26, 4:38 PM
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
Branch
oldnew1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14387
Build 18736: Run Core Tests
Build 18735: arc lint + arc unit

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.