mirror of
https://github.com/servo/servo.git
synced 2025-08-07 06:25:32 +01:00
layout: Always floor the max-content size by the min-content size (#36571)
This is a follow-up to #36518, which only addressed inline formatting contexts. However, flex formatting contexts had the same problem, so it seems safer to address it in general. Testing: this makes a WPT test pass Fixes: #36570 Signed-off-by: Oriol Brufau <obrufau@igalia.com>
This commit is contained in:
parent
594c04dc7c
commit
939355645e
4 changed files with 17 additions and 10 deletions
|
@ -279,4 +279,18 @@ pub(crate) trait ComputeInlineContentSizes {
|
|||
layout_context: &LayoutContext,
|
||||
constraint_space: &ConstraintSpace,
|
||||
) -> InlineContentSizesResult;
|
||||
|
||||
/// Returns the same result as [`Self::compute_inline_content_sizes()`], but adjusted
|
||||
/// to floor the max-content size by the min-content size.
|
||||
/// This is being discussed in <https://github.com/w3c/csswg-drafts/issues/12076>.
|
||||
fn compute_inline_content_sizes_with_fixup(
|
||||
&self,
|
||||
layout_context: &LayoutContext,
|
||||
constraint_space: &ConstraintSpace,
|
||||
) -> InlineContentSizesResult {
|
||||
let mut result = self.compute_inline_content_sizes(layout_context, constraint_space);
|
||||
let sizes = &mut result.sizes;
|
||||
sizes.max_content.max_assign(sizes.min_content);
|
||||
result
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue