REGRESSION (125-412): problem with negative margin-top and position: absolute (vre.aitg.com)

[Chris Petersen]

Steps to Reproduce: 
1. reference URL is http://vre.aitg.com
2. Click on any links along top (Service Information, Special Programs, etc)
3. Rendering is incorrect; line below text appears below "reference line" (line entirely across top of 
page, separating beige from white).

Expected Results:
"beige on beige", ie text appears above "reference" line. Appeared correctly in 1.2

Actual Results:
text blocks (Service Information, etc) appears below "reference" line

Notes:
Site primarily developed in Dreamweaver on Mac OS X 10.3. Primary browser was Safari 1.2 under 
10.3.8.

[Chris Petersen]

4/20/05 11:52 AM Chris Petersen:
I have isolated the issue to a reduced test case. The test case uses two DIV (DIV 1, DIV 2) elements. The 
problem is the DIV 2 is using absolute positioning with a negative margin value assigned.

position: absolute; margin: -24 0 0 166px;


Open the attached test case " vre_aitg.html" in Safari v412 or v312. Notice the text "Service Information" 
is displayed below the background image in DIV 1. Open "vre_aitg.html" in either Safari v125 or Firefox 
1.0.2. Notice the text is displayed above the horizontal line in the tiled background image. This is the 
correct behavior in Safari v125 and Firefox.

[Chris Petersen]

Apple Bug : <rdar://problem/4093277>

[Chris Petersen]

Created attachment 2300 [details]
Reduced test case

[Daniel Udey]

For what it's worth, this bug is still confirmed in ToT as of November 9th. It appears that WebKit is not 
applying the negative margin due to the div being position: absolute; without positioning, it works fine.

[Eric Seidel (no email)]

The test case could be made slightly more self-documentary.  Regardless, this is still clearly a bug in TOT.  
Regressions are p1s according to the guidelines.

[Darin Adler]

Created attachment 5825 [details]
an even-better reduction that shows the problem

[Darin Adler]

The problem seems to be that RenderBox::calcVerticalMargins is never called for a positioned block, so the code thinks it has a marginTop of 0.

[Darin Adler]

Created attachment 5842 [details]
patch to fix the problem (negative top margins in position: absolute case)

[Darin Adler]

Created attachment 5843 [details]
patch without the extra file (KWQListBox.mm) in it

[Maciej Stachowiak]

Comment on attachment 5843 [details]
patch without the extra file (KWQListBox.mm) in it

looks good

[Eric Seidel (no email)]

Removing Regression keyword from bugs already fixed.