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
F15916305: D17895.diff
Sun, Jun 1, 7:56 PM
Unknown Object (File)
Wed, May 28, 7:15 PM
Unknown Object (File)
Wed, May 7, 2:50 AM
Unknown Object (File)
Apr 24 2025, 2:50 AM
Unknown Object (File)
Apr 14 2025, 2:17 AM
Unknown Object (File)
Apr 11 2025, 10:09 AM
Unknown Object (File)
Apr 6 2025, 9:08 PM
Unknown Object (File)
Apr 5 2025, 8:40 PM
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