mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-07-29 20:29:18 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

LibWeb: Propagate text-decoration-* properties to anonymous wrappers

Browse source 
Fixes an issue where old prices were not displayed with strike-through
text on the PlayStation store. :^)
This commit is contained in:
Andreas Kling 2024-09-03 11:08:46 +02:00 • committed by Andreas Kling
commit 8a6c8a1c27
Notes: github-actions[bot] 2024-09-03 15:41:51 +00:00
4 changed files with 30 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

10
Tests/LibWeb/Ref/reference/text-decoration-in-anonymous-wrapper-ref.html Normal file
View file

@ -0,0 +1,10 @@
<!doctype html>
<link rel="match" href="reference/text-decoration-in-anonymous-wrapper-ref.html" />
<style type="text/css">
.strike {
text-decoration-line: line-through;
text-decoration-color: red;
text-decoration-thickness: 4px;
text-decoration-style: dotted;
}
</style><div class="strike">strike

14
Tests/LibWeb/Ref/text-decoration-in-anonymous-wrapper.html Normal file
View file

@ -0,0 +1,14 @@
<!doctype html>
<link rel="match" href="reference/text-decoration-in-anonymous-wrapper-ref.html" />
<style type="text/css">
.strike {
text-decoration-line: line-through;
text-decoration-color: red;
text-decoration-thickness: 4px;
text-decoration-style: dotted;
}
.messer-upper {
width: 0;
height: 0;
}
</style><div class="strike">strike<div class="messer-upper"></div>

BIN
Tests/LibWeb/Screenshot/images/css-background-clip-text.png
View file

Binary file not shown.
Side by side Swipe Overlay

Before

Width:  |  Height:  |  Size: 268 KiB

After

Width:  |  Height:  |  Size: 234 KiB

Before After
Before After

6
Userland/Libraries/LibWeb/Layout/Node.cpp
View file

@ -975,6 +975,12 @@ JS::NonnullGCPtr<NodeWithStyle> NodeWithStyle::create_anonymous_wrapper() const
{ {
auto wrapper = heap().allocate_without_realm<BlockContainer>(const_cast<DOM::Document&>(document()), nullptr, computed_values().clone_inherited_values()); auto wrapper = heap().allocate_without_realm<BlockContainer>(const_cast<DOM::Document&>(document()), nullptr, computed_values().clone_inherited_values());
wrapper->mutable_computed_values().set_display(CSS::Display(CSS::DisplayOutside::Block, CSS::DisplayInside::Flow)); wrapper->mutable_computed_values().set_display(CSS::Display(CSS::DisplayOutside::Block, CSS::DisplayInside::Flow));
// NOTE: These properties are not inherited, but we still have to propagate them to anonymous wrappers.
wrapper->mutable_computed_values().set_text_decoration_line(computed_values().text_decoration_line());
wrapper->mutable_computed_values().set_text_decoration_thickness(computed_values().text_decoration_thickness());
wrapper->mutable_computed_values().set_text_decoration_color(computed_values().text_decoration_color());
wrapper->mutable_computed_values().set_text_decoration_style(computed_values().text_decoration_style());
return *wrapper; return *wrapper;
} }