mirror of
https://github.com/LadybirdBrowser/ladybird.git
synced 2024-09-29 16:21:29 +00:00
LibWeb: Flexbox: Assume Block when finding max main size of flex-column
This is a hack, but it seems to do quite okay. What we should do is to find the largest size the Box could want in its main axis. To do that we have to layout the Box according to the needed LayoutMode. For flex-rows we do as requested and try to make the Box as wide as we want. However, for flex-columns we simply assume the Box is a Block and we calculate their height according to this.
This commit is contained in:
parent
ddb7402649
commit
d104885757
Notes:
sideshowbarker
2024-07-18 03:43:43 +09:00
Author: https://github.com/TobyAsE Commit: https://github.com/SerenityOS/serenity/commit/d1048857573 Pull-request: https://github.com/SerenityOS/serenity/pull/10112
|
@ -61,10 +61,12 @@ void FlexFormattingContext::run(Box& box, LayoutMode)
|
|||
return length.resolved(CSS::Length::make_px(0), box, box.containing_block()->width()).to_px(box);
|
||||
};
|
||||
auto layout_for_maximum_main_size = [&](Box& box) {
|
||||
if (is_row)
|
||||
if (is_row) {
|
||||
layout_inside(box, LayoutMode::OnlyRequiredLineBreaks);
|
||||
else
|
||||
layout_inside(box, LayoutMode::AllPossibleLineBreaks);
|
||||
return box.width();
|
||||
} else {
|
||||
return BlockFormattingContext::compute_theoretical_height(box);
|
||||
}
|
||||
};
|
||||
auto containing_block_effective_main_size = [&is_row, &main_size_is_infinite](Box& box) {
|
||||
if (is_row) {
|
||||
|
@ -330,11 +332,9 @@ void FlexFormattingContext::run(Box& box, LayoutMode)
|
|||
if (has_definite_main_size(child_box)) {
|
||||
flex_item.flex_base_size = specified_main_size_of_child_box(box, child_box);
|
||||
} else {
|
||||
layout_for_maximum_main_size(child_box);
|
||||
flex_item.flex_base_size = calculated_main_size(child_box);
|
||||
flex_item.flex_base_size = layout_for_maximum_main_size(child_box);
|
||||
}
|
||||
}
|
||||
|
||||
auto clamp_min = has_main_min_size(child_box)
|
||||
? specified_main_min_size(child_box)
|
||||
: 0;
|
||||
|
|
Loading…
Reference in a new issue