41191 - img tag not supporting scripted onclick

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[Dylan Schiemann]

The onClick event is captured properly when embedded in the tag
<img src="blank.jpg" onClick="alert('embedded click')"> or when using the 
addEventListener("click",functionName,false), but not with the simple 
document.getElementById(idOfImage).onclick = addHandler;

Comment 1

[Dylan Schiemann]

Attachment: Testcase

Comment 2

[Guillaume Filion]

I can reproduce this bug on PowerMac G3 with 160 MB RAM running Mac OS 8.6. 
Mozilla Build 20000053108. 

The test page works correctly on Explorer 5 Mac but not in Navigator 4.7.3 nor in 
Mozilla.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Is this in any DOM spec (I can't find it), or is it just a bug for IE5-parity?

Comment 4

[Dylan Schiemann]

I would think this would fall under DOM Level 0 or the Netscape 3 event model, 
which was never implemented for most elements.  This behavior is supported in 
M15 for virtually every element in a document except img.  If this is something 
worth fixing which I think it is, I guess a better testcase is needed to see 
which elements support this and which do not.

Comment 5

[Johnny Stenback (:jst)]

Joki, I played around with this a bit and it looks like we really don't handle

dynamically setting img.onclick, could you have a look?

Comment 6

[joki (gone)]

Fix in hand.  Attaching diff, not sure when this will get approval for checkin.

Comment 7

[joki (gone)]

Attachment: fix for bug

Comment 8

[Daniel Veditz [:dveditz]]

*** Bug 42693 has been marked as a duplicate of this bug. ***

Comment 9

[Daniel Veditz [:dveditz]]

Sorry, typo in the bug number. bug 42693 is not a duplicate of this bug.

Comment 10

[joki (gone)]

Adding [fixinhand] to allow easier sorting while waiting for nsbeta3 tree 
checkins.

Nominating nsbeta3.

Comment 11

[Nisheeth Ranjan]

Marking nsbeta3+...

Comment 12

[Nisheeth Ranjan]

Joki's fixes have been checked in by saari.  Marking fixed.

Comment 13

[Prashant Desale]

Nisheeth, Some how I'm not able to see the alert. I guess it is still there. I 
tried it with Win-95 too, but it does not work.
Reopening right now, if you think it is fixed please let me know like that.

Comment 14

[joki (gone)]

Well the test case seems to not have an image with it.  If you try this test 
case with a valid image it works just fine.  I don't know that we are trying to 
support events over broken images.

Comment 15

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Verified fixed on NT build around 2000-09-25.