5

I am trying to test the following JavaScript code, which is meant to keep track of the timing of user responses on a multiple choice survey: 

document.onclick = function(e) {

    var event = e || window.event;    
    var target = e.target || e.srcElement;

    //time tracking 
    var ClickTrackDate = new Date;
    var ClickData = ""; 
    ClickData = target.id + "=" + ClickTrackDate.getUTCHours() + ":" + 
        ClickTrackDate.getUTCMinutes() + ":" +
        ClickTrackDate.getUTCSeconds() +";";

    document.getElementById("txtTest").value += ClickData;
    alert(target.id); // for testing
}

Usually target.id equals to the the id of the clicked element, as you would expect, but sometimes target.id is empty, seemingly at random, any ideas?

Blizzer
  • 260
  • 4
  • 22
broccoli_soup
  • 309
  • 4
  • 14
  • 3
    You'll find usinng jquery will make this easier as it will deal with browser issues, bugs, and quirks for you. – John Conde Mar 25 '12 at 21:34

2 Answers2

9

Probably because the element that was clicked didn't have an ID. This will happen for example if you have HTML like this:

<div id="example">This is some <b>example</b> text.</div>

and someone clicks on the boldface word "example", so that the event target will be the b element instead of the div.

You could try walking up the DOM tree until you find an element that does have an ID, something like this:

var target = e.target || e.srcElement;

while (target && !target.id) {
    target = target.parentNode;
}
// at this point target either has an ID or is null
Ilmari Karonen
  • 49,047
  • 9
  • 93
  • 153
  • It does seem to be that - the radio buttons of the survey are inside table cells, and clicking just off the button but within the cell still selects it, but as in your example there is not an id for the table cell. Thanks! – broccoli_soup Mar 25 '12 at 21:54
  • Had this problem, really confusing, in my case I bound to a
  • but like you say, if I click on the containing text it works but if I click in the space outside the text it gives currentTarget null... I'm just confused over all the different event member variables and how they are supposed to point I guess :)
    • – BjornW Jun 20 '21 at 10:30