WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
55740
CSS3 multi column layout doesn't work as expected with min-height
https://bugs.webkit.org/show_bug.cgi?id=55740
Summary
CSS3 multi column layout doesn't work as expected with min-height
Kenichi Ishibashi
Reported
2011-03-03 20:27:04 PST
For details, see
http://crbug.com/65731
I've investigated briefly this problem. The cause is in RenderBlock::layoutBlock(). If the RenderBlock has multi column, it computes columnHeight by using computeLogicalHeight() beforehand the actual layout. When the block has min-height property, the columnHeight set to its value even if the height should be more larger because there is no computed height at that moment. It seems to me that we can't determine the column height at that point, but I'm not sure because I'm not familiar with layout/rendering well. Could anyone give me advice how to cope with this problem?
Attachments
Example XHTML from test forum post.
(4.39 KB, application/xhtml+xml)
2011-07-26 15:43 PDT
,
John A. Bilicki III
no flags
Details
View All
Add attachment
proposed patch, testcase, etc.
John A. Bilicki III
Comment 1
2011-07-26 15:43:50 PDT
Created
attachment 102065
[details]
Example XHTML from test forum post. The bug enforces height instead of min-height I think, this is exceptionally annoying and present in Safari 5.1 and Chrome 13 beta.
Ahmad Saleem
Comment 2
2022-12-15 14:17:55 PST
The Chrome Bug in
Comment 0
was fixed by this commit: Commit -
https://chromium.googlesource.com/chromium/src/+/79a9f512a11425ca0e55243dc10628f45484feb9
I took the above and test all test cases and Safari 16.2 passes all of them and if I see 1-1 patch and Webkit Source, there were few differences, which I tried to fix in this PR:
https://github.com/WebKit/WebKit/pull/6715
It passes all tests and does not have any existing test failures. I am not sure on how much it would fix above attached test case but just wanted to share my update before I close my PR since it is not reviewed.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug