Homec4science

Provide basic keyboard navigation support for Differential.

Authored by epriestley <git@epriestley.com> on Jun 8 2011, 20:53.

Description

Provide basic keyboard navigation support for Differential.

Summary:
ReviewBoard has a fancier version of this feature that's more granular -- the
keyboard can focus on individual changes. I think that's good and intend to
implement something similar, but this gets us a step closer and gets rid of some
of the bookkeeping stuff like making shortcuts discoverable.

(I have another brnach with Maniphest merging which also uses fatcow icons,
which is why the README seems a little out of context.)

Test Plan:
Used "j" and "k" to jump between changesets. Pressed "?" and got a list of
available shortcuts.

Reviewed By: tuomaspelkonen
Reviewers: aran, jungejason, tuomaspelkonen
CC: moskov, aran, epriestley, tuomaspelkonen
Differential Revision: 412

Details

Committed
epriestley <git@epriestley.com>Jun 9 2011, 23:55
Pushed
aubortJan 31 2017, 17:16
Parents
rPH1e5fd3a3864f: Fix Herald exception when updating a diff that has carryover CCs
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH17306b7a92bb: Provide basic keyboard navigation support for Differential. (authored by epriestley <git@epriestley.com>).Jun 9 2011, 23:55