ansaurus

Question

Why won't this Javascript Function Work?

Answer 1

+1 A:

If you're using jQuery, you can simplify your code:

$(document).keydown(e) {
    return e.keyCode == 31; // replace this with your logic
}

Koistya Navin 2009-04-05 16:27:26

This does not solve the issue, because it is the isnumberkey function that is not working, but very good to know. I am currently studying jQuery now. Thank you!

Chris B. 2009-04-05 16:32:53

Answer 2

+4 A:

Seems like your stopRKey is always returning false which could be blocking your isNumberKey.

Try re-arranging your parens.

if ((evt.keyCode == 13) && (node.type=="text"))
  $("label#enter_error").show();
{return false;}

Should be

if ((evt.keyCode == 13) && (node.type=="text")) {
  $("label#enter_error").show();
  return false;
}

Update: There are two ways of using the if statement.

You can call if with a single operation: eg.

if (true) alert('always true');
// or
if (true)
  alert('always true');

or you can call an if statement with a block of code

if (true) {
  alert('one alert');
  alert('another alert');
}

Since your parens were out of order your code was only executing the show() if the user pressed enter and then continued processing the remaining code. So return false was always called.

bendewey 2009-04-05 16:49:48

That works. Thank you very much. Could you explain how this works exactly? I thought that return false; was conditional I don't understand what it would always return false.

Chris B. 2009-04-05 16:53:41

Its really more about how the if statement functions, see my update

bendewey 2009-04-05 17:01:50

ansaurus

tags:

views:

answers:

Why won't this Javascript Function Work?

related questions