Page MenuHomePhabricator

Use prose diffs in Phriction

Authored by epriestley on Jun 6 2016, 9:01 PM.
Referenced Files
Unknown Object (File)
Thu, Nov 30, 2:38 AM
Unknown Object (File)
Sat, Nov 25, 2:35 PM
Unknown Object (File)
Thu, Nov 16, 4:28 AM
Unknown Object (File)
Oct 30 2023, 9:52 PM
Unknown Object (File)
Oct 27 2023, 12:11 AM
Unknown Object (File)
Oct 23 2023, 2:14 PM
Unknown Object (File)
Oct 23 2023, 9:25 AM
Unknown Object (File)
Oct 4 2023, 5:33 AM
"Love" token, awarded by avivey.



Ref T3353. This improves whitespace display of prose diffs and uses them in Phriction.

Test Plan

Screen Shot 2016-06-06 at 1.58.36 PM.png (1×1 px, 251 KB)

Diff Detail

rP Phabricator
Lint Not Applicable
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Use prose diffs in Phriction.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
avivey added a reviewer: avivey.
avivey added a subscriber: avivey.

This is awesome, but is it really on the way to T3353? Wouldn't we need to replace it with a view that supports inline-comments?

This revision is now accepted and ready to land.Jun 6 2016, 9:14 PM

I'll clarify T3353 shortly -- I'm not planning to, e.g., implement Google Docs.

This revision was automatically updated to reflect the committed changes.