Problem

If you focus the mod list and press space in GUI, some surprising behaviors can occur depending on exactly where you click:

• If you click the Install checkbox, then the checkbox will toggle when you press space and then again when you let go
• If you click the Upgrade checkbox, then pressing space can toggle both checkboxes

Causes

CKAN has some special code to toggle checkboxes when you press space.

• The grid itself handles checkbox toggling in KeyDown, but we were doing it in KeyPress, which happens later
• Our code fired regardless of which cell was focused, and if that cell had a checkbox, then multiple toggles would happen at different times
• Our code only ever toggled the Install checkbox, so if you click an upgradable row and press space, it would try to remove the mod rather than upgrade it

Changes

Now the behavior is more consistent and less surprising.

• Now we handle space in KeyDown for consistency with the built in behavior
• If one of the checkbox cells has the focus, we let the grid handle it
• If there's an Update checkbox, we toggle it preferentially, otherwise we toggle the Install checkbox

Fixes #2534.