Page MenuHomePhabricator
Differential D21277

Render inline comment suggestions as real diffs
ClosedPublic
Actions

Authored by epriestley on May 20 2020, 4:10 PM.
Tags
None
Referenced Files
F19771323: D21277.diff
Fri, Feb 20, 12:40 AM
F19767274: D21277.id.diff
Thu, Feb 19, 2:42 PM
F19558557: D21277.diff
Jan 30 2026, 11:50 PM
F19485656: D21277.id50671.diff
Jan 8 2026, 3:10 AM
F19469721: D21277.id50662.diff
Jan 5 2026, 3:04 AM
F19236597: D21277.diff
Dec 21 2025, 7:19 PM
F19012303: D21277.id.diff
Nov 22 2025, 12:07 PM
F19002298: D21277.diff
Nov 21 2025, 5:23 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rP10f241352dce: Render inline comment suggestions as real diffs
Summary

Ref T13513. When rendering an inline suggestion for display, use highlighting and diffing.

Test Plan

Screen Shot 2020-05-20 at 9.02.43 AM.png (232×957 px, 34 KB)

Diff Detail

Repository
rP Phabricator
Branch
inline11
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/infrastructure/diff/view/PHUIDiffInlineCommentDetailView.php:550XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 24481
Build 33736: Run Core Tests
Build 33735: arc lint + arc unit

Event Timeline

epriestley created this revision.May 20 2020, 4:10 PM
Harbormaster completed remote builds in B24481: Diff 50662.May 20 2020, 4:12 PM
epriestley requested review of this revision.May 20 2020, 4:12 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 20 2020, 9:27 PM
Closed by commit rP10f241352dce: Render inline comment suggestions as real diffs (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP10f241352dce: Render inline comment suggestions as real diffs.

Revision Contents
Changeset List

PathSize
resources/
celerity/
src/
applications/
differential/
parser/
5 lines
render/
39 lines
infrastructure/
diff/
inline/
10 lines
query/
5 lines
view/
27 lines
webroot/
rsrc/
css/
application/
differential/
24 lines

Diff 50662

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/differential/parser/DifferentialChangesetParser.php

Loading...
View Options

src/applications/differential/render/DifferentialChangesetOneUpRenderer.php

Loading...
View Options

src/infrastructure/diff/inline/PhabricatorDiffInlineCommentContext.php

Loading...
View Options

src/infrastructure/diff/query/PhabricatorDiffInlineCommentQuery.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffInlineCommentDetailView.php

Loading...
View Options

webroot/rsrc/css/application/differential/phui-inline-comment.css

Loading...
Phabricator · Built by Phacility