firefox return key

[raicha]

Hello everyone,
I am very new to this forum as well as to javascript. I have following code for enter key event on my page. its working in all browsers except firefox. I have searched online a lot but nothing has worked and finally I have thought to get some help from the experts.

<script>
function checkKey()
{
if (window.event.keyCode == 13)
{
loadUrl();
}
return true;
}
</script>


onkeydown="checkKey();return true;"

any help is much appreciated.

Thank you.

Kind Regards.

[Philip M]

<script type = "text/javascript">

document.onkeydown = function(ev) {	
var key;
ev = ev || event;
key = ev.keyCode;
alert ("Keycode = " + key);

if (key == 13) {
alert ("You pressed the ENTER key");
}

}
</script>

All advice is supplied packaged by intellectual weight, and not by volume. Contents may settle slightly in transit.

[Arbitrator]

Originally Posted by raicha

its working in all browsers except firefox.

window.event is specific to Internet Explorer 8 and earlier. Every other browser (including Internet Explorer 9) supports DOM2 Events, which means that you need to pass the event when the function is called.

The following code does that while still supporting older versions of Internet Explorer:

onkeydown="if (typeof(event) === 'object') { checkKey(event); } else { checkKey(window.event); }"

function checkKey(event) {
	if (event.keyCode === 13) {
		loadUrl();
	}
}

I removed instances of return true; since that code isn't doing anything.

[raicha]

@Philip M and Kitkat 007. when I used your code, it produced alert on every key press and shown every key code. when I removed alert ("Keycode = " + key); from it, it worked fine. But whenever I press enter button it runs the function doesn't matter where my cursor is. that is undesirable. I wanted to run the function only by pressing enter key if cursor is in text box. Thank you for helping out though.

@Arbitrator Thank you very much. your code worked like a charm. It is exactly what I am looking for. I was using IE7 by mistake therefore previous code worked in IE too. thanks for letting me know about DOM2 support. you have made my night as I am going to sleep really well after solving this puzzle.
Thank you.

Kind Regards.

[felgall]

Originally Posted by Arbitrator

onkeydown="if (typeof(event) === 'object') { checkKey(event); } else { checkKey(window.event); }"

Since the IE window.event can simply be referenced as event you can simplify that down to:

onkeydown="checkKey(event)";

[Philip M]

Originally Posted by raicha

@Philip M and Kitkat 007. when I used your code, it produced alert on every key press and shown every key code. when I removed alert ("Keycode = " + key); from it, it worked fine.

Well, yes. I would expect that you realised that the alert is simply for testing purposes.

You did not say that you wanted it to work only when the cursor is in a textbox. You said "I have following code for enter key event on my page."

<input type = "text" id = "textfield" size = "20" onkeyup = "chk(event)">

<script type = "text/javascript">

function chk(ev) {
var key;
ev = ev || event;
key = ev.keyCode;
if (key == 13) {
alert ("You pressed the Enter key");
return false;
}
}

</script>

[Arbitrator]

Originally Posted by felgall

Since the IE window.event can simply be referenced as event you can simplify that down to:

onkeydown="checkKey(event)";

Good catch.

I guess that means that Philip M's code can be simplified too:

function chk(event) {
	if (event.keyCode == 13) {
		alert("You pressed the Enter key");
		return false;
	}
}

[raicha]

Thank you everyone for great help. everyone's code is working really charm but I can only use one of them though. I didn't know that members of this forum are so much helpful. Really pleased to be a part of this community.
Thanks everyone.