system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-07-22 02:26:11 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

LibWeb: Take padding into account when resolving border radii

Browse source 
Before this change, we were resolving border radii values based on
content box + border widths only, ignoring padding.
This commit is contained in:
Andreas Kling 2024-01-07 16:19:47 +01:00
parent 402de2985d
commit ca57e40350
3 changed files with 19 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
Tests/LibWeb/Ref/border-radius-with-padding.html Normal file
View file

@ -0,0 +1,10 @@
<!doctype html><head>
<link rel="match" href="reference/border-radius-with-padding-ref.html" />
<style type="text/css">
div {
border: 1px solid black;
border-radius: 32px;
padding: 25px;
width: 0px;
}
</style></head><body><div>

8
Tests/LibWeb/Ref/reference/border-radius-with-padding-ref.html Normal file
View file

@ -0,0 +1,8 @@
<!doctype html><style type="text/css">
div {
border: 1px solid black;
border-radius: 32px;
width: 50px;
height: 50px;
}
</style><div>

3
Userland/Libraries/LibWeb/Layout/LayoutState.cpp
View file

@ -285,8 +285,7 @@ void LayoutState::resolve_border_radii()
if (paintable && is<Painting::PaintableBox>(*paintable)) {
auto& paintable_box = static_cast<Painting::PaintableBox&>(*paintable);
CSSPixelRect const content_rect { 0, 0, used_values.content_width(), used_values.content_height() };
auto border_rect = content_rect.inflated(used_values.border_top, used_values.border_right, used_values.border_bottom, used_values.border_left);
CSSPixelRect const border_rect { 0, 0, used_values.border_box_width(), used_values.border_box_height() };
auto const& border_top_left_radius = node.computed_values().border_top_left_radius();
auto const& border_top_right_radius = node.computed_values().border_top_right_radius();