Page MenuHomePhabricator
Differential D21250

Store inline comment offset information and show it when highlighting comments
ClosedPublic
Actions

Authored by epriestley on May 13 2020, 11:55 PM.
Tags
None
Referenced Files
F18850028: D21250.id50605.diff
Thu, Oct 30, 1:55 PM
F18785629: D21250.diff
Tue, Oct 14, 8:03 AM
F18782215: D21250.id50600.diff
Mon, Oct 13, 12:56 AM
F18779417: D21250.id.diff
Sat, Oct 11, 1:54 PM
F18752412: D21250.diff
Oct 4 2025, 3:11 PM
F18112342: D21250.diff
Aug 12 2025, 4:26 PM
F18110792: D21250.id.diff
Aug 12 2025, 12:01 AM
F18105916: D21250.diff
Aug 10 2025, 8:34 PM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13513: Make schema changes to inlines to prepare for new inline features
Commits
rP2f5398796e02: Store inline comment offset information and show it when highlighting comments
Summary

Ref T13513. When a user selects a text range and uses "New Inline Comment" to create a comment directly from a range, store the offset information alongside the comment.

When hovering the comment, highlight the original range.

Test Plan

character-ranges.gif (712×772 px, 1 MB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.May 13 2020, 11:55 PM
Harbormaster completed remote builds in B24417: Diff 50600.May 13 2020, 11:56 PM
epriestley requested review of this revision.May 13 2020, 11:56 PM
This revision was not accepted when it landed; it landed in state Needs Review.May 14 2020, 12:21 AM
Closed by commit rP2f5398796e02: Store inline comment offset information and show it when highlighting comments (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP2f5398796e02: Store inline comment offset information and show it when highlighting comments.

Revision Contents
Changeset List

PathSize
resources/
celerity/
src/
applications/
celerity/
postprocessor/
2 lines
infrastructure/
diff/
4 lines
interface/
18 lines
view/
2 lines
webroot/
rsrc/
css/
application/
differential/
24 lines
js/
application/
diff/
4 lines
279 lines
49 lines

Diff 50605

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/celerity/postprocessor/CelerityDefaultPostprocessor.php

Loading...
View Options

src/infrastructure/diff/PhabricatorInlineCommentController.php

Loading...
View Options

src/infrastructure/diff/interface/PhabricatorInlineComment.php

Loading...
View Options

src/infrastructure/diff/view/PHUIDiffInlineCommentView.php

Loading...
View Options

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

Loading...
View Options

webroot/rsrc/js/application/diff/DiffChangeset.js

Loading...
View Options

webroot/rsrc/js/application/diff/DiffChangesetList.js

Loading...
View Options

webroot/rsrc/js/application/diff/DiffInline.js

Loading...
Phabricator · Built by Phacility