Differential D17859

Move most Differetial keyboard shortcuts into DiffChangesetList
ClosedPublic
Actions

Authored by epriestley on May 9 2017, 5:07 PM.

Tags

None

Referenced Files

	F19080083: D17859.id43037.diff
	Tue, Dec 2, 12:00 AM

	F19072955: D17859.id43055.diff
	Mon, Dec 1, 12:18 AM

	F19040967: D17859.id43037.diff
	Wed, Nov 26, 2:09 PM

	F19034278: D17859.id43055.diff
	Tue, Nov 25, 1:51 PM

	F19011138: D17859.diff
	Sat, Nov 22, 7:53 AM

	F18974783: D17859.id43055.diff
	Sat, Nov 15, 11:59 PM

	F18951259: D17859.id43055.diff
	Wed, Nov 12, 3:43 AM

	F18920959: D17859.id.diff
	Nov 9 2025, 1:32 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12616: Modernize display code for inline comments

Commits

rP588a66c04d03: Move most Differetial keyboard shortcuts into DiffChangesetList

Summary

Ref T12616. This moves most keyboard shortcuts into DiffChangesetList. It breaks some shortcuts that I plan to restore later, noted in T12616 (toggle file, edit inline, reply to inline), since I think ripping them out now and rebuilding them in a little bit will make things much simpler.

Test Plan

Used j, k, n, p, J, K shortcuts to navigate a revision.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 9 2017, 5:07 PM

Harbormaster completed remote builds in B16878: Diff 42952.May 9 2017, 5:08 PM

chad accepted this revision.May 9 2017, 5:09 PM

This revision is now accepted and ready to land.May 9 2017, 5:09 PM

epriestley mentioned this in T12616: Modernize display code for inline comments.May 9 2017, 6:15 PM

epriestley mentioned this in D17894: Move inline replies to new code and remove DifferentialInlineEditor.May 15 2017, 11:49 PM

Rebase this on top of D17894; I'll restore missing shortcuts soon.

Harbormaster completed remote builds in B16964: Diff 43037.May 15 2017, 11:59 PM

Closed by commit rP588a66c04d03: Move most Differetial keyboard shortcuts into DiffChangesetList (authored by epriestley, committed by epriestley). · Explain WhyMay 16 2017, 1:23 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in rP41379f39debb: Move inline replies to new code and remove DifferentialInlineEditor.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

47 lines

1 line

src/

applications/

differential/

controller/

DifferentialRevisionViewController.php

1 line

view/

DifferentialChangesetListView.php

10 lines

diffusion/

controller/

DiffusionCommitController.php

2 lines

webroot/

rsrc/

js/

application/

diff/

DiffChangeset.js

82 lines

DiffChangesetList.js

162 lines

differential/

behavior-keyboard-nav.js

Diff 43055

resources/celerity/map.php

Loading...

resources/celerity/packages.php

Loading...

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...

src/applications/differential/view/DifferentialChangesetListView.php

Loading...

src/applications/diffusion/controller/DiffusionCommitController.php

Loading...

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

Loading...

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

Loading...

webroot/rsrc/js/application/differential/behavior-keyboard-nav.js

Loading...