jquery: do this on keyup, unless the person is in a textarea or input

Question

my script works but i don't understand how to make it NOT launch the functions when in a textarea/input and those keys are pressed. aka: launch the event when the user presses that key, unless the user is in a textarea/input.

$('body').keyup(function (event) {

var direction = null;
if (event.keyCode == 37) {
  $('#wrapper').fadeOut(500)
} else if (event.keyCode == 39) {
        $('html,body').animate({scrollTop: $('body').offset().top}, {duration: 1500, easing: 'easeInOutQuart'}
        )
return false;    
}
     })

fantactuka · Accepted Answer · 2010-09-24T14:30:09.607

10

Just check event.target:

$('body').keyup(function(event) {
    if ($(event.target).is(':not(input, textarea)')) {
       ...
    }
});

In this case you will still have only one event handler (attached to the body) but it will filter for elements that recieves the event

edited Sep 24 '10 at 14:30

answered Sep 24 '10 at 14:24

fantactuka

3,298
19
29

Why assign the event to the input in the first place if you never want to handle it? – djdd87 Sep 24 '10 at 14:34
1

Here we assign event to the body, not to the input. And later just filter with input and textarea.. It's much better then assign event to all elements except inputs – fantactuka Sep 24 '10 at 14:45

score 1 · Answer 2 · answered Sep 24 '10 at 14:21

1

Try:

$('body *:not(textarea,input)').keyup(function (event) {

});

answered Sep 24 '10 at 14:21

Thariama

50,002
13
138
166

1

Such selection is incorrect since it will add as much event handlers as much elements you have on the page. – fantactuka Sep 24 '10 at 14:42
almost right (textareas are not included), but this will work – Thariama Sep 24 '10 at 15:04

score 1 · Answer 3 · answered Sep 24 '10 at 14:23

1

$('body :not(textarea,input[type=text])').keyup(function (event) {

or

$('body').not('textarea,input[type=text]').keyup(function (event) {

answered Sep 24 '10 at 14:23

Dave Thieben

5,388
2
28
38

This one has same problem as Thariama's solution – fantactuka Sep 24 '10 at 14:44
Just imagine that page has 10000 elements (not too much). Will you assign event to each element? It does not makes sense – fantactuka Sep 24 '10 at 15:01
1

I suppose it's a good design decision for the OP. however, not what he was asking about. – Dave Thieben Sep 24 '10 at 15:04
I'm just suggesting to think about performance of your solution. Whatever he asked for. – fantactuka Sep 24 '10 at 15:41

jquery: do this on keyup, unless the person is in a textarea or input

3 Answers3