Bug 5813 - positioned blocks don't update when resizing the window vertically
Summary: positioned blocks don't update when resizing the window vertically
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 412
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Beth Dakin
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-11-23 15:33 PST by Timothy Hatcher
Modified: 2006-02-08 23:48 PST (History)
6 users (show)

See Also:


Attachments
Test Case (396 bytes, text/html)
2005-11-23 15:34 PST, Timothy Hatcher
no flags Details
Always reposition the root's positioned children (7.35 KB, patch)
2006-02-07 13:39 PST, mitz
hyatt: review-
Details | Formatted Diff | Diff
Always reposition the root's positioned children (10.22 KB, patch)
2006-02-08 06:24 PST, mitz
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Timothy Hatcher 2005-11-23 15:33:25 PST
A div with an absolute position that is pinned to the bottom will not update as the window resizes only 
vertically. Any horizontal resize will make the block adjust it's size/position. The block will only resize 
once the border touches the frame edge.
Comment 1 Timothy Hatcher 2005-11-23 15:34:03 PST
Created attachment 4786 [details]
Test Case

resize this test case vertically
Comment 2 mitz 2006-02-05 05:33:14 PST
*** Bug 7081 has been marked as a duplicate of this bug. ***
Comment 3 mitz 2006-02-07 12:58:22 PST
This bug is similar to bug 6881. The fix there was to assure that positioned blocks are repositioned if their container's height changed, since that's what affects their position.

The rendering root is a special case since it uses its availableHeight (essentially the viewport's height) for positioning instead of height. However, layoutBlock can't check for changes to availableHeight, since it's already changed by the time it's called. I'm going to propose a patch that always repositions if isRoot().
Comment 4 mitz 2006-02-07 13:39:03 PST
Created attachment 6335 [details]
Always reposition the root's positioned children
Comment 5 Dave Hyatt 2006-02-07 19:30:41 PST
Comment on attachment 6335 [details]
Always reposition the root's positioned children

You missed flexbox again.  Need to patch it and then also include it in the test case.
Comment 6 Dave Hyatt 2006-02-07 19:33:48 PST
I guess you need 2 test cases, since you have to use the root element to test. :)
Comment 7 mitz 2006-02-08 06:24:19 PST
Created attachment 6341 [details]
Always reposition the root's positioned children

Oops. Applied to flexbox and added it to the test.
Comment 8 Dave Hyatt 2006-02-08 22:39:37 PST
Comment on attachment 6341 [details]
Always reposition the root's positioned children

r=me
Comment 9 Timothy Hatcher 2006-02-08 23:48:18 PST
Landed in r12712.