Bug 9347

Summary:

Border on root (<html>) element causes elements with position: absolute to layout incorrectly

Product:

WebKit

Reporter:

Sam Weinig <sam>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

ian, mrowe

Priority:

Version:

420+

Hardware:

Mac

OS:

OS X 10.4

Attachments:

Description	Flags
test case in Strict mode	none
test case in Quirks mode	none
Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms)	mjs: review+

Sam Weinig

Reported 2006-06-07 15:19:42 PDT

Moz has a similar bug in their bugzilla https://bugzilla.mozilla.org/show_bug.cgi?id=243519

Attachments
test case in Strict mode (478 bytes, text/html) 2006-06-07 15:20 PDT, Sam Weinig	no flags	Details
test case in Quirks mode (461 bytes, text/html) 2006-06-07 15:21 PDT, Sam Weinig	no flags	Details
Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) (4.63 KB, patch) 2006-10-16 01:17 PDT, Dave Hyatt	mjs: review+	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Sam Weinig

Comment 1 2006-06-07 15:20:39 PDT

Created attachment 8757 [details] test case in Strict mode

Sam Weinig

Comment 2 2006-06-07 15:21:01 PDT

Created attachment 8758 [details] test case in Quirks mode

Dave Hyatt

Comment 3 2006-09-12 23:58:58 PDT

This is a stunning change from CSS2. I do not believe it is compatible with the existing Web either. We should think very carefully before fixing this.

Dave Hyatt

Comment 4 2006-10-16 01:17:29 PDT

Created attachment 11109 [details] Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) Here we go. A followup improvement I can make after this goes in is to eliminate the RenderLayer for the root element.

Dave Hyatt

Comment 5 2006-10-16 01:27:07 PDT

Disregard my dramatic comment. I didn't quite understand the 2.0 -> 2.1 spec change. I do now, and it's not that scary. :)

Maciej Stachowiak

Comment 6 2006-10-16 01:30:15 PDT

Comment on attachment 11109 [details] Patch that fixes the containing block for positioned elements to be the RenderView ("initial containing block" in CSS terms) r=me

Dave Hyatt

Comment 7 2006-10-16 01:34:37 PDT

Fixed.

David Kilzer (:ddkilzer)

Comment 8 2006-10-17 02:48:27 PDT

(In reply to comment #7) > Fixed. Tests in r17067, fix in r17068.

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