Bug 9347 - Border on root (<html>) element causes elements with position: absolute to layout incorrectly
Summary: Border on root (<html>) element causes elements with position: absolute to la...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-07 15:19 PDT by Sam Weinig
Modified: 2006-10-17 02:48 PDT (History)
2 users (show)

See Also:

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 an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Weinig 2006-06-07 15:19:42 PDT 
Moz has a similar bug in their bugzilla https://bugzilla.mozilla.org/show_bug.cgi?id=243519
Comment 1 Sam Weinig 2006-06-07 15:20:39 PDT 
Created attachment 8757 [details]
test case in Strict mode
Comment 2 Sam Weinig 2006-06-07 15:21:01 PDT 
Created attachment 8758 [details]
test case in Quirks mode
Comment 3 Dave Hyatt 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.
Comment 4 Dave Hyatt 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.
Comment 5 Dave Hyatt 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. :)
Comment 6 Maciej Stachowiak 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
Comment 7 Dave Hyatt 2006-10-16 01:34:37 PDT 
Fixed.
Comment 8 David Kilzer (:ddkilzer) 2006-10-17 02:48:27 PDT 
(In reply to comment #7)
> Fixed.

Tests in r17067, fix in r17068.