From 655f054b4fa2ed6239f61071ad1c44e85548711c Mon Sep 17 00:00:00 2001 From: Kenneth Myhra Date: Sun, 20 Feb 2022 10:03:10 +0100 Subject: [PATCH] LibWeb: Add default padding around contents of text elements This patch adds a default padding around the contents of text elements. It adds these defaults to the existing style attribute in 'HTMLInputElement::create_shadow_tree_if_needed()'. Use a default padding for text elements: - padding-top and padding-bottom: 1px - padding-left and padding-right: 2px These values seems to align with what other browsers do. --- Userland/Libraries/LibWeb/HTML/HTMLInputElement.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/Libraries/LibWeb/HTML/HTMLInputElement.cpp b/Userland/Libraries/LibWeb/HTML/HTMLInputElement.cpp index 9bc18e12ded..03bce5f0b4b 100644 --- a/Userland/Libraries/LibWeb/HTML/HTMLInputElement.cpp +++ b/Userland/Libraries/LibWeb/HTML/HTMLInputElement.cpp @@ -172,7 +172,7 @@ void HTMLInputElement::create_shadow_tree_if_needed() if (initial_value.is_null()) initial_value = String::empty(); auto element = document().create_element(HTML::TagNames::div); - element->set_attribute(HTML::AttributeNames::style, "white-space: pre"); + element->set_attribute(HTML::AttributeNames::style, "white-space: pre; padding-top: 1px; padding-bottom: 1px; padding-left: 2px; padding-right: 2px"); m_text_node = adopt_ref(*new DOM::Text(document(), initial_value)); m_text_node->set_always_editable(true); m_text_node->set_owner_input_element({}, *this);