views:

546

answers:

3

I have a table row which contains a textbox and it has an onclick which displays a javascript calendar... I am adding rows to the table with textbox, but i dont know how to attach onclick event to the javascript generated textbox...

<input  class="date_size_enquiry" type="text" autocomplete="off"
onclick="displayDatePicker('attendanceDateadd1');" size="21" readonly="readonly"    
maxlength="11" size="11"  name="attendanceDateadd1" id="attendanceDateadd1" 
value="" onblur="per_date()" onchange="fnloadHoliday(this.value);">

And my javascript generates a textbox,

    var cell2 = row.insertCell(1);
    cell2.setAttribute('align','center')
    var el = document.createElement('input');
    el.className = "date_size_enquiry";
    el.type = 'text';
    el.name = 'attendanceDateadd' + iteration;
    el.id = 'attendanceDateadd' + iteration;
    el.onClick = //How to call the fuction displayDatePicker('attendanceDateadd1');
    e1.onblur=??
    e1.onchange=??
    cell2.appendChild(el);
+3  A: 

Like this:

var el = document.createElement('input');
...
el.onclick = function() {
    displayDatePicker('attendanceDateadd1');
};

BTW: Be careful with case sensitivity in the DOM. It's "onclick", not "onClick".

Asaph
@ Asaph it didnt work `el.onClick = function(){alert(el.id);};`
chandru_cp
@chandru_cp: Don't capitalize the "c" in "onclick". It's a lower case "c". See the last line of my answer.
Asaph
@Asaph that worked...
chandru_cp
@chandru_cp: Why did you unaccept this answer?
Asaph
@Asaph sorry one of my friend used my system
chandru_cp
+2  A: 
el.onclick = function(){
  displayDatePicker(this.id);
};
Shawn Steward
+1  A: 

Taking your example, I think you want to do this:

el.onclick = function() { displayDatePicker(el.id); };

The only trick is to realise why you need to wrap your displayDatePicker call in the function() { ... } code. Basically, you need to assign a function to the onclick property, however in order to do this you can't just do el.onclick = displayDatePicker(el.id) since this would tell javascript to execute the displayDatePicker function and assign the result to the onclick handler rather than assigning the function call itself. So to get around this you create an anonymous function that in turn calls your displayDatePicker. Hope that helps.

Alconja