Firefox problem... return false not working?

[grande]

So, here's what's up. I have this function:

function keyclick(e, entity) {
	var keynum;
	if (window.event) {    // IE
		keynum = e.keyCode; }
	else {                 // Opera, Netscape, Firefox
		keynum = e.which; }
	if (keynum == 13) {
		alert(entity);
		pushData(entity);
		alert("Pushing Data");
		return false;
	}
}

As you can see, it calls the pushData function. When fake-stepping through (via alerts), pushdata gets called, and everything *should* be fine. BUT, when the Enter key is pressed, the page reloads. I thought I had taken care of this with the return false (works in IE), but Firefox still reloads the page.

Help?

-------------------------
Call me barely Impressive Captain.

 

[BillyRayPreachersSon]

As you can see, it calls the pushData function.

Must be foggy in here - I can't see anywhere how or where you are calling this.

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]

 

[BillyRayPreachersSon]

I'm referring to "keyclick", btw - not "pushData".

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]

 

[grande]

function keyclick(e, entity) {
    var keynum;
    if (window.event) {    // IE
        keynum = e.keyCode; }
    else {                 // Opera, Netscape, Firefox
        keynum = e.which; }
    if (keynum == 13) {
        alert(entity);
********pushData(entity);********
        alert("Pushing Data");
        return false;
    }
}

Keyclick is called here:

OnKeyDown="javascript:return keyclick(event, 'AuditTable_lblConstructionSC');"

-------------------------
Call me barely Impressive Captain.

 

[Aarem]

try removing the javascript: code prefix - they don't go in on* attributes.

 

[BillyRayPreachersSon]

Does Fx give they keycode as 13? Does this alert correctly? Can you give some more detail about the element with the onkeydown? Perhaps the whole element? At present you only seem to give away the minimum asked for rather than something marginally richer.

Dan



Coedit Limited - Delivering standards compliant, accessible web solutions

[tt]Dan's Page [blue]@[/blue] Code Couch

http://www.codecouch.com/dan/

[/tt]