Page MenuHomePhabricator
Differential D17895

Restore the Differential "edit" and "reply" keyboard shortcuts
ClosedPublic
Actions

Authored by epriestley on May 16 2017, 12:33 AM.
Tags
None
Referenced Files
F15502598: D17895.id43039.diff
Mon, Apr 14, 2:17 AM
F15489559: D17895.id43038.diff
Fri, Apr 11, 10:09 AM
F15475636: D17895.id43056.diff
Sun, Apr 6, 9:08 PM
F15473240: D17895.id.diff
Sat, Apr 5, 8:40 PM
F15468707: D17895.diff
Fri, Apr 4, 4:46 AM
F15424107: D17895.id43056.diff
Sat, Mar 22, 8:13 PM
F15396302: D17895.diff
Mar 16 2025, 1:22 PM
F15395715: D17895.id43038.diff
Mar 16 2025, 9:11 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T12616: Modernize display code for inline comments
Commits
rP2fb1edfeb198: Restore the Differential "edit" and "reply" keyboard shortcuts
Summary

Ref T12616. This makes "edit" and "reply" work again.

Test Plan

Used "e" and "r" to edit and reply.

Also used them in bogus ways and got useful UI feedback.

Diff Detail

Repository
rP Phabricator
Branch
inline14
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16966
Build 22680: Run Core Tests
Build 22679: arc lint + arc unit

Revision Contents
Changeset List

PathSize
resources/
celerity/
48 lines
src/
applications/
differential/
view/
8 lines
webroot/
rsrc/
js/
application/
diff/
9 lines
55 lines
25 lines

Diff 43039

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/differential/view/DifferentialChangesetListView.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...
Phabricator · Built by Phacility