beenull/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2024-11-22 07:30:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

LibWeb: Use correct specifier to pad font language override value

Browse source
This commit is contained in:
Tim Ledbetter 2024-11-18 16:37:35 +00:00 committed by Tim Flynn
parent 96a35767b6
commit 7fe110225b
Notes: github-actions[bot] 2024-11-18 22:39:04 +00:00 
Author: https://github.com/tcl3
Commit: https://github.com/LadybirdBrowser/ladybird/commit/7fe110225b7
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/2424
Reviewed-by: https://github.com/trflynn89
3 changed files with 12 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
Libraries/LibWeb/CSS/Parser/Parser.cpp
View file

@ -5777,7 +5777,7 @@ RefPtr<CSSStyleValue> Parser::parse_font_language_override_value(TokenStream<Com
}
transaction.commit();
if (length < 4)
return StringStyleValue::create(MUST(String::formatted("{<4}", string_value)));
return StringStyleValue::create(MUST(String::formatted("{:<4}", string_value)));
return string;
}

1
Tests/LibWeb/Text/expected/css/font-language-override-short.txt Normal file
View file

@ -0,0 +1 @@
PASS (didn't crash)

10
Tests/LibWeb/Text/input/css/font-language-override-short.html Normal file
View file

@ -0,0 +1,10 @@
<!DOCTYPE html>
<script src="../include.js"></script>
<script>
test(() => {
const element = document.createElement('div');
element.style.fontLanguageOverride = '"eng"';
document.body.appendChild(element);
println("PASS (didn't crash)");
});
</script>