Bug 9100

Summary:

Absolutely positioned replaced elements with all non-auto values are rendered incorrectly

Product:

WebKit

Reporter:

Sam Weinig <sam>

Component:

Layout and Rendering

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

Keywords:

Regression

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
test case of the horizontal	none
test case of the vertical	none
patch	hyatt: review+

Sam Weinig

Reported 2006-05-24 15:34:14 PDT

In a regression from the shipping Safari, if an absolutly posioned replaced object (i.e. and image) has top, bottom, margin-top, margin-bottom with all values other than auto (the same is true for the horizontal equivilants), it ends up giving incorrect rendering by making top and bottom go to 0.

Attachments
test case of the horizontal (509 bytes, text/html) 2006-05-24 15:42 PDT, Sam Weinig	no flags	Details
test case of the vertical (507 bytes, text/html) 2006-05-24 15:43 PDT, Sam Weinig	no flags	Details
patch (4.60 KB, patch) 2006-05-25 07:59 PDT, Sam Weinig	hyatt: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Sam Weinig

Comment 1 2006-05-24 15:42:52 PDT

Created attachment 8523 [details] test case of the horizontal

Sam Weinig

Comment 2 2006-05-24 15:43:21 PDT

Created attachment 8524 [details] test case of the vertical

Sam Weinig

Comment 3 2006-05-24 15:50:18 PDT

The patch for this is pretty trivial but it will be much easier once bug 9085 is finished an landed.

Sam Weinig

Comment 4 2006-05-25 07:59:22 PDT

Created attachment 8539 [details] patch Scratch the last comment. I fixed the patch for bug 9085 to not interfere and was therefore able to make this patch. It very simply adds an else clause to cover the case when all the values are not auto.

Dave Hyatt

Comment 5 2006-05-29 23:03:52 PDT

Comment on attachment 8539 [details] patch r=me

Note You need to log in before you can comment on or make changes to this bug.