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
F13103190: D21277.diff
Sat, Apr 27, 9:33 AM
Unknown Object (File)
Thu, Apr 25, 6:10 PM
Unknown Object (File)
Tue, Apr 23, 6:51 PM
Unknown Object (File)
Tue, Apr 23, 6:51 PM
Unknown Object (File)
Tue, Apr 23, 6:51 PM
Unknown Object (File)
Fri, Apr 12, 2:41 AM
Unknown Object (File)
Fri, Mar 29, 12:41 PM
Unknown Object (File)
Fri, Mar 29, 11:27 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