[Fix] Keyboard nav issues

[erin-doyle]

This is a combination of a few things:

• making the change requested in Issue Keyboard navigation issues #557 where we need the Enter key to be able to close the Keyboard Shortcuts Panel when open
• I found that the event.key value for the spacebar key is not 'Space' at all 😳 but is either ' ' or 'Spacebar' on older browsers
• Added the throttle enhancements to the DayPicker that had been made to the DateInput in Pull Request Fix React warning when Events are referenced later #682 (fixing Issue Synthetic event warning onKeyDown #514)

[coveralls]

Coverage increased (+0.009%) to 85.533% when pulling 4465651 on fix_keyboard_nav_issues into f0b6f2f on master.

[ljharb]

I’m not sure what the semver label should be for this PR.

[ljharb]

Can this be a Set instead of an array?

[erin-doyle]

Yep I can do that.

[erin-doyle]

Done. (was going to say "all set" 😝)

[erin-doyle]

I think this is a patch. These are all just fixes, there isn't any breaking changes and nothing new really. Now the Keyboard Shortcuts Panel will close using the Enter key where as before it was only Spacebar and Esc but that was really a miss, so I consider this fixing that not adding something new.

[coveralls]

Coverage increased (+0.009%) to 85.533% when pulling 0cb4ea1 on fix_keyboard_nav_issues into f0b6f2f on master.

[ljharb]

LGTM!

[erin-doyle]

@majapw just want to bump this if you have a sec to take a look now that you're back from vacay. ;)

[ljharb]

(it'll need a rebase also)

[erin-doyle]

Pushed rebase. 👍

[coveralls]

Coverage increased (+0.009%) to 86.257% when pulling db515e7 on fix_keyboard_nav_issues into 9780545 on master.

[majapw]

Looking now! :)

[majapw]

Looks wonderful! Thank you.

I found that the event.key value for the spacebar key is not 'Space' at all 😳 but is either ' ' or 'Spacebar' on older browsers

I too had this issue. :P