RESOLVED FIXED 3662
onClick event for TD element doesn't fire unless its contents are clicked
https://bugs.webkit.org/show_bug.cgi?id=3662
Summary onClick event for TD element doesn't fire unless its contents are clicked
Sebastien Boisvert
Reported 2005-06-22 20:09:20 PDT
An onClick event specified for a TD element will not fire unless its contents are clicked, instead of simply clicking anywhere within the cell. Steps to reproduce: 1- load attached testcase in browser 2- click anywhere in table cell, except on text 3- click text in table cell Actual result: alert triggered by onclick event only pops up when the cell's contents are clicked Expected result: alert should popup regardless of where within the cell the mouse click occurs Additional Information: Works as expected in Firefox, Camino, IE/Win, IE/Mac
Attachments
testcase (300 bytes, text/html)
2005-06-22 20:09 PDT, Sebastien Boisvert
no flags
patch to fix this (1.85 KB, patch)
2005-08-14 23:55 PDT, Darin Adler
mjs: review+
Sebastien Boisvert
Comment 1 2005-06-22 20:09:54 PDT
Created attachment 2562 [details] testcase
Chris Petersen
Comment 2 2005-06-22 22:10:51 PDT
Confirming with TOT Webkit and Safari 2.0 (v412).
Frido Ferdinand
Comment 3 2005-08-09 06:02:26 PDT
Also occurs with the mouseover event
Dave Hyatt
Comment 4 2005-08-09 12:11:18 PDT
*** Bug 4339 has been marked as a duplicate of this bug. ***
Darin Adler
Comment 5 2005-08-14 19:44:13 PDT
The behavior of this is not entirely consistent. I can construct test cases where the onclick event sometimes fires and sometimes does not fire. I'll attach one soon.
Darin Adler
Comment 6 2005-08-14 23:55:53 PDT
Created attachment 3395 [details] patch to fix this
Maciej Stachowiak
Comment 7 2005-08-17 23:02:07 PDT
Comment on attachment 3395 [details] patch to fix this r=me can this be tested in an automated way?
Darin Adler
Comment 8 2005-08-19 09:43:45 PDT
I don't see a way to exercise the hit testing code from JavaScript.
David Harrison
Comment 9 2005-08-19 16:37:28 PDT
Added originally attached test case to manual_tests.
Note You need to log in before you can comment on or make changes to this bug.