RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-27 01:17:35 +00:00
Code Issues Activity Actions

LibWeb: Find font that has whitespace glyph in first_available_font()

Browse source 
Fixes https://github.com/SerenityOS/serenity/issues/22853
This commit is contained in:
Aliaksandr Kalenik 2024-01-21 22:53:54 +01:00 • committed by Alexander Kalenik
parent 6685e4cb61
commit 1294cfb55c
3 changed files with 25 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

14
Tests/LibWeb/Ref/first-available-font-should-include-whitespace.html Normal file
View file

@ -0,0 +1,14 @@
<!doctype html><link rel="match" href="reference/first-available-font-should-include-whitespace-ref.html" /><style>
* {
outline: 1px solid black;
font-family: HashFont, SerenitySans;
}
div {
width: min-content;
}
@font-face {
font-family: 'HashFont';
src: url('assets/HashSans.woff');
unicode-range: U+0;
}
</style><div class="test">hello friends

9
Tests/LibWeb/Ref/reference/first-available-font-should-include-whitespace-ref.html Normal file
View file

@ -0,0 +1,9 @@
<!doctype html><style>
* {
outline: 1px solid black;
font-family: SerenitySans;
}
div {
width: min-content;
}
</style><div class="test">hello friends

5
Userland/Libraries/LibWeb/Layout/Node.h
View file

@ -306,9 +306,8 @@ inline NodeWithStyle* Node::parent()
inline Gfx::Font const& NodeWithStyle::first_available_font() const inline Gfx::Font const& NodeWithStyle::first_available_font() const
{ {
// https://drafts.csswg.org/css-fonts/#first-available-font // https://drafts.csswg.org/css-fonts/#first-available-font
// FIXME: Should be be the first font for which the character U+0020 (space) instead of // First font for which the character U+0020 (space) is not excluded by a unicode-range
// any first font in the list return computed_values().font_list().font_for_code_point(' ');
return computed_values().font_list().first();
} }
} }