I have a function to send ments on enter (and not send on shift+enter):

$(msg).keypress(function (e) {
    if (event.keyCode == 13 && event.shiftKey) {
        event.stopPropagation();
    } else if (e.which == 13) {
        // ...
    }
});

It is working on Chrome but not in Firefox.

I have a function to send ments on enter (and not send on shift+enter):

$(msg).keypress(function (e) {
    if (event.keyCode == 13 && event.shiftKey) {
        event.stopPropagation();
    } else if (e.which == 13) {
        // ...
    }
});

It is working on Chrome but not in Firefox.

• javascript
• jquery
• events

• 1 What exactly is "not working"? The ment is not sent? Or it is sent even if shift is pressed? – Felix Kling Commented Oct 27, 2015 at 17:03
• 1 jQuery normalizes keycodes using event.which but you would need e.which – charlietfl Commented Oct 27, 2015 at 17:05
• 2 The argument you are accepting is e but you are accessing event. – sdgluck Commented Oct 27, 2015 at 17:05
• 1 Why are you using event? – Derek 朕會功夫 Commented Oct 27, 2015 at 17:06
• 1 And of course, in the one case you're checking shiftKey and in the other, you aren't. – T.J. Crowder Commented Oct 27, 2015 at 17:06

2 Answers 2

It's failing in Firefox because you've tried to reference the IE-specific global event variable (which Chrome also provides, as a bone thrown to IE-specific code), as sdgluck pointed out. Firefox doesn't have that, and so the code throws an error.

Use the argument your handler is passed (e in your example), and use which, which jQuery will normalize for you.

I think the correct way of doing it is to use this:

$(msg).keypress(function(e) {

    var key = e.keyCode || e.which;

    if(key == 13 && e.shiftKey) {
        e.stopPropagation();
    }
});