Also noticed that we are attaching to the keydown event for other keyboard matters. Probably should be consistent, either all keyup or all keydown (unless there is a good reason to have them different).