set onClick property to a dynamically generated row

[nikeshkm]

Hi all,

i ma adding a row dynamically and one of the cell contains an img icon...
to this img icon i want to set its onclick property.
i did it using the following:
imgElement.onClick = clickFunc();

but this calls the function when the cell is being appended to the dynamically added table row....
How can i make it working.

Here is the code

var seventhCell = newRow.insertCell(6);
var imgElement = document.createElement("img");
imgElement.src = "Delete.jpg";
imgElement.onClick = clickFunc(); //here is the problem.
seventhCell.appendChild(imgElement);


pls suggest something.
thnx in advance.

[acoder]

Welcome to TSDN.

Remove the parentheses. What you are currently doing is running the function and setting the result to the onclick. What you want is to set the actual function to the onclick by just using the name:

imgElement.onClick = clickFunc;

[ssh]

imgElement.onClick = clickFunc(); //here is the problem.

hi!

You can try the below code .. it should work ..

imgElement.onClick = function(){clickFunc()};

Regards,
ssh

[nikeshkm]

hey
thnx all
but both r not working...donno y
as per the code

var seventhCell = newRow.insertCell(6);
var imgElement = document.createElement("img");
imgElement.src = "Delete.jpg";
imgElement.onClick = clickFunc(); //here is the problem.
seventhCell.appendChild(imgElement);

"Delete.jpg" is present in the same folder as the JSP..
but still the image is not shown..
also i tried the sattic path but not working..

as per ur suggestion i tried both ways to set onclick property
but both r not wotking..

[acoder]

Try onclick with a lower-case "c".

[nikeshkm]

tht was so silly of me...
thnx a lot....it worked...
but the image is not yet displayed...
wot cud b the reason..???

[nikeshkm]

well its done..
i gave the wrong path...
thnx all bye

[acoder]

Glad you got it working.