LadybirdBrowser/ladybird
1
1
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-09-20 17:58:18 +03:00
Code Issues Packages Projects Releases Wiki Activity

LibWeb: Parse each unquoted font-family name as a single CustomIdentSV

Browse Source 
Previously we made StringStyleValues from these, but once we start
actually quoting StringStyleValues when serializing them, this will
break the font-family serialization.
This commit is contained in:
Sam Atkins 2023-09-12 12:44:59 +01:00 committed by Sam Atkins
parent d1e542999c
commit 77ae510319
Notes: sideshowbarker 2024-07-17 06:29:49 +09:00 
Author: https://github.com/AtkinsSJ
Commit: https://github.com/SerenityOS/serenity/commit/77ae510319
Pull-request: https://github.com/SerenityOS/serenity/pull/21043
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Userland/Libraries/LibWeb/CSS/Parser/Parser.cpp
View File

@ -4146,7 +4146,8 @@ RefPtr<StyleValue> Parser::parse_font_family_value(TokenStream<ComponentValue>&
if (current_name_parts.is_empty())
return nullptr;
(void)tokens.next_token(); // Comma
font_families.append(StringStyleValue::create(MUST(String::join(' ', current_name_parts))));
// This is really a series of custom-idents, not just one. But for the sake of simplicity we'll make it one.
font_families.append(CustomIdentStyleValue::create(MUST(String::join(' ', current_name_parts))));
current_name_parts.clear();
// Can't have a trailing comma
if (!tokens.has_next_token())
@ -4158,7 +4159,8 @@ RefPtr<StyleValue> Parser::parse_font_family_value(TokenStream<ComponentValue>&
}
if (!current_name_parts.is_empty()) {
font_families.append(StringStyleValue::create(MUST(String::join(' ', current_name_parts))));
// This is really a series of custom-idents, not just one. But for the sake of simplicity we'll make it one.
font_families.append(CustomIdentStyleValue::create(MUST(String::join(' ', current_name_parts))));
current_name_parts.clear();
}