When the search box is not present (e.g.,
https://docs.racket-lang.org/demo-m1/index.html),
pressing "S" will result in a JS error. This PR fixes the problem.
Note that semantically it makes more sense to give an ID to the
search box as we know exactly what search box we want.
In non QWERTY keyboard, pressing S won't focus the search bar.
This commit fixes the problem.
Note that `keyCode` is not supported in IE8 already, so the fact
that `key` is not supported in IE8 too doesn't really matter:
the PR doesn't cause browser compatibility for the features to degrade.
1. Focus on the search box for either "s" or "S"
... accepting only "s" makes sense to me, but the comment said it
accepted "S" and well why not
2. Look for a "keyup" event instead of key press, so that pressing "s"
ONLY focuses on the box and does not focus-and-write-the-letter-"s"
"keyup" events apparently don't have a useful `charCode` field so this
PR looks at the `keyCode` field instead
