Page MenuHomePhabricator
Differential D16062

Use prose diffs in Phriction
ClosedPublic
Actions

Authored by epriestley on Jun 6 2016, 9:01 PM.
Tags
None
Referenced Files
F19679287: D16062.id38648.diff
Sun, Feb 8, 6:04 PM
F19039493: D16062.diff
Nov 26 2025, 8:13 AM
F19026031: D16062.diff
Nov 24 2025, 9:50 AM
F18872386: D16062.id.diff
Nov 5 2025, 3:56 AM
F18869034: D16062.diff
Nov 4 2025, 9:41 AM
F18847312: D16062.diff
Oct 29 2025, 11:57 PM
F18816124: D16062.diff
Oct 21 2025, 5:03 AM
F18809474: D16062.id.diff
Oct 19 2025, 2:20 PM
View All Files
Subscribers
avivey
Tokens
"Love" token, awarded by avivey.

Details

Reviewers
avivey
chad
Maniphest Tasks
T3353: Make Phriction Document changes reviewable
Commits
rP02877c600a75: Use prose diffs in Phriction
Summary

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

Repository
rP Phabricator
Branch
prose4
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12519
Build 15876: Run Core Tests
Build 15875: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38648.Jun 6 2016, 9:01 PM
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.
epriestley added a task: T3353: Make Phriction Document changes reviewable.
Herald edited edge metadata. · View Herald TranscriptJun 6 2016, 9:01 PM
avivey awarded a token.Jun 6 2016, 9:11 PM
avivey accepted this revision.Jun 6 2016, 9:14 PM
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
epriestley added a comment.Jun 6 2016, 9:33 PM

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

Closed by commit rP02877c600a75: Use prose diffs in Phriction (authored by epriestley, committed by epriestley). · Explain WhyJun 6 2016, 9:33 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
12 lines
src/
applications/
phriction/
controller/
65 lines
webroot/
rsrc/
css/
application/
differential/
8 lines

Diff 38648

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/phriction/controller/PhrictionDiffController.php

Loading...
View Options

webroot/rsrc/css/application/differential/changeset-view.css

Loading...
Phabricator · Built by Phacility