summaryrefslogtreecommitdiff
path: root/Userland/Libraries
diff options
context:
space:
mode:
authorSam Atkins <atkinssj@serenityos.org>2022-02-16 16:32:18 +0000
committerAndreas Kling <kling@serenityos.org>2022-02-16 21:47:53 +0100
commit48edaa2085d06bc8c129a70bcfd69021e3adf555 (patch)
tree0082344f936316fc0eabb8e3e56915ca68668d00 /Userland/Libraries
parent5d2ce68f5363b0d3e4f797b060cbe66d947c6716 (diff)
downloadserenity-48edaa2085d06bc8c129a70bcfd69021e3adf555.zip
LibWeb: Remove content-size hack from SVGFormattingContext
Everything functions fine without it. :^)
Diffstat (limited to 'Userland/Libraries')
-rw-r--r--Userland/Libraries/LibWeb/Layout/SVGFormattingContext.cpp12
1 files changed, 1 insertions, 11 deletions
diff --git a/Userland/Libraries/LibWeb/Layout/SVGFormattingContext.cpp b/Userland/Libraries/LibWeb/Layout/SVGFormattingContext.cpp
index af4fb3fe83..8d273aa905 100644
--- a/Userland/Libraries/LibWeb/Layout/SVGFormattingContext.cpp
+++ b/Userland/Libraries/LibWeb/Layout/SVGFormattingContext.cpp
@@ -1,5 +1,6 @@
/*
* Copyright (c) 2021, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, Sam Atkins <atkinssj@serenityos.org>
*
* SPDX-License-Identifier: BSD-2-Clause
*/
@@ -7,7 +8,6 @@
#include <AK/Format.h>
#include <LibWeb/Layout/SVGFormattingContext.h>
#include <LibWeb/Layout/SVGGeometryBox.h>
-#include <LibWeb/Layout/SVGSVGBox.h>
namespace Web::Layout {
@@ -22,12 +22,6 @@ SVGFormattingContext::~SVGFormattingContext()
void SVGFormattingContext::run(Box& box, LayoutMode)
{
- // FIXME: This formatting context is basically a total hack.
- // It works by computing the united bounding box of all <path>'s
- // within an <svg>, and using that as the size of this box.
-
- Gfx::FloatRect total_bounding_box;
-
box.for_each_in_subtree_of_type<SVGBox>([&](auto& descendant) {
if (is<SVGGeometryBox>(descendant)) {
auto& geometry_box = static_cast<SVGGeometryBox&>(descendant);
@@ -40,14 +34,10 @@ void SVGFormattingContext::run(Box& box, LayoutMode)
geometry_box.set_offset(bounding_box.top_left());
geometry_box.set_content_size(bounding_box.size());
-
- total_bounding_box = total_bounding_box.united(path.bounding_box());
}
return IterationDecision::Continue;
});
-
- box.set_content_size(total_bounding_box.size());
}
}