dmaax/ladybird
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

LibWeb: Avoid min-content sizing for fitting fit-content widths

Browse source 
Measure max-content before min-content for definite fit-content widths.
When the max-content width is no larger than stretch-fit, the clamp
resolves to max-content and min-content cannot affect the result.

Leave height sizing unchanged because its intrinsic measurements depend
on the width passed into the layout.
This commit is contained in:
Andreas Kling 2026-06-08 17:39:20 +02:00 committed by Andreas Kling
parent d7a5fca98b
commit a92418c812
1 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
Libraries/LibWeb/Layout/FormattingContext.cpp
View file

@ -2045,9 +2045,11 @@ CSSPixels FormattingContext::calculate_fit_content_width(Layout::Box const& box,
// If the available space in a given axis is definite, equal to clamp(min-content size, stretch-fit size,
// max-content size) (i.e. max(min-content size, min(max-content size, stretch-fit size))).
if (available_space.width.is_definite()) {
return max(calculate_min_content_width(box),
min(calculate_stretch_fit_width(box, available_space.width),
calculate_max_content_width(box)));
auto stretch_fit_width = calculate_stretch_fit_width(box, available_space.width);
auto max_content_width = calculate_max_content_width(box);
if (max_content_width <= stretch_fit_width)
return max_content_width;
return max(calculate_min_content_width(box), stretch_fit_width);
}
// When sizing under a min-content constraint, equal to the min-content size.