mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-08-04 07:09:47 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 10

LibWeb: Disable ligatures if text-rendering is set to optimizeSpeed

Browse source 
If `font-variant-ligatures` is set to `normal` and `text-rendering` is
set to `optimizeSpeed` then all ligatures are now disabled.
This commit is contained in:
Tim Ledbetter 2025-06-27 15:26:39 +01:00 • committed by Sam Atkins
commit d7bdbeb446
Notes: github-actions[bot] 2025-06-27 15:52:53 +00:00
1 changed files with 13 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

18
Libraries/LibWeb/Layout/InlineLevelIterator.cpp
View file

@ -211,15 +211,20 @@ HashMap<StringView, u8> InlineLevelIterator::shape_features_map() const
// 6.4 https://drafts.csswg.org/css-fonts/#font-variant-ligatures-prop
auto ligature_or_null = computed_values.font_variant_ligatures();
auto disable_all_ligatures = [&]() {
features.set("liga"sv, 0);
features.set("clig"sv, 0);
features.set("dlig"sv, 0);
features.set("hlig"sv, 0);
features.set("calt"sv, 0);
};
if (ligature_or_null.has_value()) {
auto ligature = ligature_or_null.release_value();
if (ligature.none) {
// Specifies that all types of ligatures and contextual forms covered by this property are explicitly disabled.
features.set("liga"sv, 0);
features.set("clig"sv, 0);
features.set("dlig"sv, 0);
features.set("hlig"sv, 0);
features.set("calt"sv, 0);
disable_all_ligatures();
} else {
switch (ligature.common) {
case Gfx::FontVariantLigatures::Common::Common:
@ -275,6 +280,9 @@ HashMap<StringView, u8> InlineLevelIterator::shape_features_map() const
break;
}
}
} else if (computed_values.text_rendering() == CSS::TextRendering::Optimizespeed) {
// AD-HOC: Disable ligatures if font-variant-ligatures is set to normal and text rendering is set to optimize speed.
disable_all_ligatures();
} else {
// A value of normal specifies that common default features are enabled, as described in detail in the next section.
features.set("liga"sv, 1);