Bug 8081 - REGRESSION: Drop-down menu has gap at top
Summary: REGRESSION: Drop-down menu has gap at top
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://drew.edu
Keywords: InRadar, Regression
Depends on:
Reported: 2006-03-30 09:17 PST by John Muccigrosso
Modified: 2006-03-31 11:14 PST (History)
1 user (show)

See Also:

Reduction without problem (15.61 KB, text/html)
2006-03-30 09:17 PST, John Muccigrosso
no flags Details
Reduction (15.59 KB, text/html)
2006-03-30 09:19 PST, John Muccigrosso
no flags Details
Further reduction (389 bytes, text/html)
2006-03-30 13:37 PST, mitz
no flags Details
Proposed patch (6.11 KB, patch)
2006-03-30 15:02 PST, mitz
darin: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Muccigrosso 2006-03-30 09:17:02 PST
Mousing over green items at top center of page produces a drop-down menu which has a gap between the text and the first item in the drop-down menu, which is otherwise complete.
Comment 1 John Muccigrosso 2006-03-30 09:17:48 PST
Created attachment 7394 [details]
Reduction without problem

Commented out most of original document
Comment 2 John Muccigrosso 2006-03-30 09:18:43 PST
Comment on attachment 7394 [details]
Reduction without problem

Made an additional change to reduction to eliminate bug.
Comment 3 John Muccigrosso 2006-03-30 09:19:14 PST
Created attachment 7395 [details]

Commented out extraneous scripts and text.
Comment 4 mitz 2006-03-30 09:42:17 PST
Confirmed. This is a regression in TOT.
Comment 5 mitz 2006-03-30 13:37:11 PST
Created attachment 7404 [details]
Further reduction
Comment 6 mitz 2006-03-30 13:41:15 PST
Comment on attachment 7404 [details]
Further reduction

Forgot to say that this should look like a green square with a 1px red border. The absolutely-positioned block seems to add up the height of the rows above its row twice.
Comment 7 mitz 2006-03-30 15:02:11 PST
Created attachment 7407 [details]
Proposed patch
Comment 8 Darin Adler 2006-03-30 20:53:38 PST
Comment on attachment 7407 [details]
Proposed patch

Is this the only place that needs an isTableRow() check like this? Is there a better way to write the check than isTableRow()?

Comment 9 Beth Dakin 2006-03-31 11:14:32 PST
I committed this fix. Also, this same problem is in Radar as <rdar://problem/4496510>