RESOLVED FIXED 9085
Absolutely positioned objects with 'height': auto and 'top': and 'bottom': non-auto should not be content based 
https://bugs.webkit.org/show_bug.cgi?id=9085
Summary Absolutely positioned objects with 'height': auto and 'top': and 'bottom': no...
Sam Weinig
Reported 2006-05-23 18:02:05 PDT
Absolutly positioned objects that have a value other than auto for 'top' and 'bottom' and a value of auto for 'height' should not compute the height to be based on the content (like is done if either or both of 'top' and 'bottom' are auto). Instead the value should be calculated using the constraint equation for absolutely position objects.
Attachments
test case (894 bytes, text/html)
2006-05-23 18:02 PDT, Sam Weinig 		no flags
Details
patch (13.68 KB, patch)
2006-05-23 18:26 PDT, Sam Weinig 		no flags
DetailsFormatted DiffDiff
patch2 (8.72 KB, patch)
2006-05-24 18:14 PDT, Sam Weinig 		hyatt: review+
DetailsFormatted DiffDiff
another test case of interest (641 bytes, text/html)
2006-05-27 01:05 PDT, Maciej Stachowiak 		no flags
Details
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2006-05-23 18:02:56 PDT
Created attachment 8506 [details] test case
Sam Weinig
Comment 2 2006-05-23 18:26:19 PDT
Created attachment 8507 [details] patch Moves overflow code to after all the height calculations have been made for calcAbsoluteVertical and entirely removes it from the replaced version where it should never be used. Also moves the calculation of containerheight and containerwidth to sepertate functions inorder to share the code.
Sam Weinig
Comment 3 2006-05-24 18:14:54 PDT
Created attachment 8530 [details] patch2 More targeted version of the patch.
Maciej Stachowiak
Comment 4 2006-05-27 01:05:09 PDT
Created attachment 8569 [details] another test case of interest Hyatt mentioned that the bug in this additional test case is likely to be related. In this case, percentage height for a child of an absolute positioned element with height unspecified but top and bottom specified is not handled correctly.
Sam Weinig
Comment 5 2006-05-27 16:03:21 PDT
Added bug 9137 to track the issue in maciej's test case.
Dave Hyatt
Comment 6 2006-05-29 23:11:09 PDT
Comment on attachment 8530 [details] patch2 r=me
Beth Dakin
Comment 7 2006-05-30 13:44:20 PDT
I committed this patch.
David Kilzer (:ddkilzer)
Comment 8 2006-07-01 08:58:47 PDT
*** Bug 7174 has been marked as a duplicate of this bug. ***
Note You need to log in before you can comment on or make changes to this bug.