Remove empty 'gi' handler; make code work for different prefixes.

The prefix is now set based on the `event.which` number, and is
used in a conditional in the `handlePrefixShortcuts` method to
determine which shortcuts are called by that particular prefix.

This would allow for easily handling other prefix keys, by adding
more conditions or a case statement in the `handlePrefixShortcuts`
method.

Also removed the empty handler for the 'gi' shortcut, since that
isn't being used yet.
Move `event.preventDefault()` out of helper method.

The `event.prevendDefault()` was being called twice in the helper
method `handlePrefixedShortcuts`, and was using a closure to access
the `event` variable.  It makes more sense to prevent the default
behavior of the event earlier, in the place where
`handlePrefixedShortcuts` is called.
Tidy up comments; fix typos and style.

Fixes a capitalization error, adds hanging indentation, and fixes
typo in declaring DataType.
Rename 'gg' and 'G' handlers for clarity.

The methods were named to follow the pattern of other methods in this
file; `focusFirstFocusable`, but since they both also cause scrolling
they were renamed to describe both moving focus and scrolling;
`scrollAndFocusTop`.
Refactor the initial setting of focus and set it with 'j' and 'gg'.

The `moveFocus` method is now doing one of four types of movements,
so it shouldn't also be tasked with placing the initial focus if
there is no `focused` element yet.

This commit adds a helper method and calls it if focus is not placed yet
for either the 'j' or 'gg' shortcuts.

Issues:
 * Before each movement shortcut, we want to set the focus if it hasn't
 already been set.  The helper method to do this can be called as the
 optional first step inside of `moveFocus` rather than calling it every
 time that `moveFocus` fails.
Refactors the setting of focus when no focus is set.

When using keyboard shortcuts, the focus should be set the first time
a movement keyboard shortcut is used.  For example, 'G' should both
scroll to the bottom and set the focus on the last element if no focus
had been set already.

Since checking and setting the focus is a prerequisite for any movement,
this commit adds a helper method to get/set the focus and uses the
helper method at the start of the `moveFocus` method.
Move magic numbers into variables and make consistent.

The ms times passed for the scrolling and setting timeout for
prefixed keyboard shortcuts were hard-coded in.  Better to have
those at the top for consistency and easy configuration.
Clean up style and put 'gg' and 'G' near 'j' and 'k' in modal.

Style fixes:
 * spaces inside of `{}`s
 * add missing space after comma
 * chain var assignment and align `=`

Also changes order of shortcuts in help modal to make more sense.