dmauro / Keypress

A keyboard input capturing utility in which any key can be a modifier key.
http://dmauro.github.io/Keypress/
Apache License 2.0
3.18k stars 313 forks source link

"meta /" doesn't work on windows #7

Closed xiaoli closed 11 years ago

xiaoli commented 11 years ago
keypress.combo('meta /', function(event) {
  // show help dialog ...
});

The problem on windows is if we holding "ctrl" then press down "/", it won't work. But reversely it works.

xiaoli commented 11 years ago
keypress.sequence_combo('meta /', function(event) {
  // show help dialog ...
});

This bit works as promised.

dmauro commented 11 years ago

I'll take a look into this first chance I get. Thanks. Just in case, which Windows are you running and do other meta key combos work for you, such as meta h or some other random combo?

xiaoli commented 11 years ago

Both don't work on Win7 & IE9/Chrome:

keypress.combo('meta /', function(event) {
  // show help dialog ...
});
keypress.combo('meta s', function(event) {
  // save current state ...
});

Also failed with:

keypress.combo('ctrl /', function(event) {
  // show help dialog ...
});
keypress.combo('ctrl s', function(event) {
  // save current state ...
});

If pressing "s" or "/" + "ctrl", then it works. But "ctrl" + "s" or "/" get nothing.

Mac OS X & Safari/Chrome/Firefox don't have this issue.

dmauro commented 11 years ago

Okay thank you, that's very helpful. I'll track this down.

dmauro commented 11 years ago

Is Windows running natively or are you running via Parallels or something?

xiaoli commented 11 years ago

natively running on Dell desktop/laptop machines.

dmauro commented 11 years ago

Hey, this should be working now with 1.0.4. Thanks for your help and sorry it took me so long to get around to this.

tiye commented 11 years ago

@Xiaoli Works now, both ctrl / and / ctrl on IE8 and IE9. ctrl (keyup and quickly press) / also triggers that event.

dmauro commented 11 years ago

Great, thanks for confirming! (the keyup and then press / is a little worrisome though, I'll check that out)