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
Unknown Object (File)
Wed, Jun 10, 1:00 AM
Unknown Object (File)
Tue, Jun 2, 6:59 AM
Unknown Object (File)
Apr 3 2026, 12:45 PM
Unknown Object (File)
Feb 20 2026, 12:40 AM
Unknown Object (File)
Feb 19 2026, 2:42 PM
Unknown Object (File)
Jan 30 2026, 11:50 PM
Unknown Object (File)
Jan 8 2026, 3:10 AM
Unknown Object (File)
Jan 5 2026, 3:04 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