Bug 3662

Summary:

onClick event for TD element doesn't fire unless its contents are clicked

Product:

WebKit

Reporter:

Sebastien Boisvert <sebastienboisvert>

Component:

Tables

Assignee:

Dave Hyatt <hyatt>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

frido.ferdinand, maksim, will.mccullough

Priority:

Version:

412

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
testcase	none
patch to fix this	mjs: review+

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	Details
patch to fix this (1.85 KB, patch) 2005-08-14 23:55 PDT, Darin Adler	mjs: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

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.