Bug 55740 - CSS3 multi column layout doesn't work as expected with min-height
Summary: CSS3 multi column layout doesn't work as expected with min-height
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: CSS (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Nobody
URL: http://crbug.com/65731
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-03 20:27 PST by Kenichi Ishibashi
Modified: 2023-02-22 03:57 PST (History)
7 users (show)

See Also:


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

Note You need to log in before you can comment on or make changes to this bug.
Description Kenichi Ishibashi 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?
Comment 1 John A. Bilicki III 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.
Comment 2 Ahmad Saleem 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.