Page MenuHomePhabricator
Differential D17894

Move inline replies to new code and remove DifferentialInlineEditor
ClosedPublic
Actions

Authored by epriestley on May 15 2017, 11:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 22 2024, 11:46 PM
Unknown Object (File)
Oct 19 2024, 11:09 AM
Unknown Object (File)
Oct 18 2024, 5:55 AM
Unknown Object (File)
Sep 11 2024, 1:12 AM
Unknown Object (File)
Sep 11 2024, 1:09 AM
Unknown Object (File)
Sep 11 2024, 1:08 AM
Unknown Object (File)
Sep 11 2024, 1:05 AM
Unknown Object (File)
Aug 27 2024, 12:05 PM
View All 99 Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12616: Modernize display code for inline comments
Commits
rP41379f39debb: Move inline replies to new code and remove DifferentialInlineEditor
Summary

Ref T12616. This moves "reply" to the new stuff and deletes DifferentialInlineEditor, which no longer does anything.

(This breaks some keyboard shortcuts, but I'll rebase D17859 shortly.)

Test Plan

Replied to inlines; things seemed to work properly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
resources/
celerity/
75 lines
3 lines
src/
infrastructure/
diff/
13 lines
webroot/
rsrc/
js/
application/
diff/
12 lines
9 lines
61 lines
differential/
93 lines

Diff 43054

View Options

resources/celerity/map.php

Loading...
View Options

resources/celerity/packages.php

Loading...
View Options

src/infrastructure/diff/PhabricatorInlineCommentController.php

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...
View Options

webroot/rsrc/js/application/differential/DifferentialInlineCommentEditor.js

Loading...
View Options

webroot/rsrc/js/application/differential/behavior-edit-inline-comments.js

Loading...
Phabricator · Built by Phacility