From 1e526af430a5028302688996348f6aff2e4e93d5 Mon Sep 17 00:00:00 2001 From: Andi Gallo Date: Fri, 26 May 2023 13:47:40 +0000 Subject: [PATCH] LibWeb: Fix width calculation for floating replaced elements The path for floating, replaced elements must not fall through to the path taken for floating, non-replaced elements. The former works like inline replaced elements, while the latter uses a completely different algorithm which doesn't account for intrinsic ratio. Falling through overrides the correct value computed by the former. Fixes #19061. --- .../expected/resolve-height-of-containing-block.txt | 6 +++--- .../Layout/input/resolve-height-of-containing-block.html | 2 +- .../Libraries/LibWeb/Layout/BlockFormattingContext.cpp | 8 +++++++- 3 files changed, 11 insertions(+), 5 deletions(-) diff --git a/Tests/LibWeb/Layout/expected/resolve-height-of-containing-block.txt b/Tests/LibWeb/Layout/expected/resolve-height-of-containing-block.txt index d8d1dedc68..92f0f5f6fa 100644 --- a/Tests/LibWeb/Layout/expected/resolve-height-of-containing-block.txt +++ b/Tests/LibWeb/Layout/expected/resolve-height-of-containing-block.txt @@ -8,10 +8,10 @@ Viewport <#document> at (0,0) content-size 800x600 children: not-inline BlockContainer at (8,16) content-size 1280x800 children: not-inline BlockContainer <(anonymous)> at (8,16) content-size 1280x0 children: inline TextNode <#text> - BlockContainer
at (8,16) content-size 1280x600 children: not-inline + BlockContainer
at (8,16) content-size 1280x400 children: not-inline BlockContainer <(anonymous)> at (8,16) content-size 1280x0 children: inline TextNode <#text> - ImageBox at (88,16) content-size 1200x600 floating children: not-inline + ImageBox at (488,16) content-size 800x400 floating children: not-inline TextNode <#text> BlockContainer

at (8,16) content-size 1280x17.46875 children: inline line 0 width: 37.21875, height: 17.46875, bottom: 17.46875, baseline: 13.53125 @@ -20,7 +20,7 @@ Viewport <#document> at (0,0) content-size 800x600 children: not-inline TextNode <#text> BlockContainer <(anonymous)> at (8,49.46875) content-size 1280x0 children: inline TextNode <#text> - BlockContainer <(anonymous)> at (8,616) content-size 1280x0 children: inline + BlockContainer <(anonymous)> at (8,416) content-size 1280x0 children: inline TextNode <#text> BlockContainer <(anonymous)> at (8,816) content-size 784x0 children: inline TextNode <#text> diff --git a/Tests/LibWeb/Layout/input/resolve-height-of-containing-block.html b/Tests/LibWeb/Layout/input/resolve-height-of-containing-block.html index b8aa340683..abaf5d0cd8 100644 --- a/Tests/LibWeb/Layout/input/resolve-height-of-containing-block.html +++ b/Tests/LibWeb/Layout/input/resolve-height-of-containing-block.html @@ -8,7 +8,7 @@ } .foo div { - height: 600px; + height: 400px; } .foo div img { diff --git a/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp b/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp index 51941ab76c..cf50bf9d3f 100644 --- a/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp +++ b/Userland/Libraries/LibWeb/Layout/BlockFormattingContext.cpp @@ -137,10 +137,16 @@ void BlockFormattingContext::compute_width(Box const& box, AvailableSpace const& // FIXME: This const_cast is gross. const_cast(replaced).prepare_for_replaced_layout(); compute_width_for_block_level_replaced_element_in_normal_flow(replaced, remaining_available_space); - // NOTE: We don't return here. + if (box.is_floating()) { + // 10.3.6 Floating, replaced elements: + // https://www.w3.org/TR/CSS22/visudet.html#float-replaced-width + return; + } } if (box.is_floating()) { + // 10.3.5 Floating, non-replaced elements: + // https://www.w3.org/TR/CSS22/visudet.html#float-width compute_width_for_floating_box(box, available_space); return; }