Bug 55740 – CSS3 multi column layout doesn't work as expected with min-height

• Component: CSS
• Status: NEW

[Kenichi Ishibashi]

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?

[John A. Bilicki III]

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]

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.