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
F12834629: D16817.diff
Thu, Mar 28, 2:42 PM
Unknown Object (File)
Wed, Mar 27, 9:44 PM
Unknown Object (File)
Wed, Mar 27, 9:44 PM
Unknown Object (File)
Feb 19 2024, 2:47 AM
Unknown Object (File)
Feb 7 2024, 8:38 AM
Unknown Object (File)
Feb 3 2024, 12:43 PM
Unknown Object (File)
Jan 25 2024, 10:51 AM
Unknown Object (File)
Dec 30 2023, 9:29 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
Branch
oldnew1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14394
Build 18747: Run Core Tests
Build 18746: 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.