From ae93aeb414334e5bb58517279783c01b95c74ca9 Mon Sep 17 00:00:00 2001
From: Sam Atkins <atkinssj@serenityos.org>
Date: Tue, 15 Feb 2022 16:42:00 +0000
Subject: LibWeb: Give `<svg>` elements a size again

This replaces the unused width() and height() methods. The size now
defaults to 100% by 100% as in the spec.
---
 Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

(limited to 'Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp')

diff --git a/Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp b/Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp
index f7583076a0..df0e3b937a 100644
--- a/Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp
+++ b/Userland/Libraries/LibWeb/SVG/SVGSVGElement.cpp
@@ -5,6 +5,7 @@
  */
 
 #include <LibGfx/Painter.h>
+#include <LibWeb/CSS/Parser/Parser.h>
 #include <LibWeb/CSS/StyleComputer.h>
 #include <LibWeb/DOM/Document.h>
 #include <LibWeb/DOM/Event.h>
@@ -24,14 +25,21 @@ RefPtr<Layout::Node> SVGSVGElement::create_layout_node(NonnullRefPtr<CSS::StyleP
     return adopt_ref(*new Layout::SVGSVGBox(document(), *this, move(style)));
 }
 
-unsigned SVGSVGElement::width() const
+void SVGSVGElement::apply_presentational_hints(CSS::StyleProperties& style) const
 {
-    return attribute(HTML::AttributeNames::width).to_uint().value_or(300);
-}
-
-unsigned SVGSVGElement::height() const
-{
-    return attribute(HTML::AttributeNames::height).to_uint().value_or(150);
+    // Width defaults to 100%
+    if (auto width_value = parse_html_length(document(), attribute("width"))) {
+        style.set_property(CSS::PropertyID::Width, width_value.release_nonnull());
+    } else {
+        style.set_property(CSS::PropertyID::Width, CSS::PercentageStyleValue::create(CSS::Percentage { 100 }));
+    }
+
+    // Height defaults to 100%
+    if (auto height_value = parse_html_length(document(), attribute("height"))) {
+        style.set_property(CSS::PropertyID::Height, height_value.release_nonnull());
+    } else {
+        style.set_property(CSS::PropertyID::Height, CSS::PercentageStyleValue::create(CSS::Percentage { 100 }));
+    }
 }
 
 void SVGSVGElement::parse_attribute(FlyString const& name, String const& value)
-- 
cgit v1.2.3