Commit 2539a566 authored by Harald Sitter's avatar Harald Sitter 🏳️‍🌈
Browse files

fonts: honor & present system defaults

previously we'd pretend that a missing value meant our "plasma-ish"
defaults would apply but that is utterly false. when no hitting is set,
no hinting is set. this can happen when the system default fontconfigs
don't set up any hinting.

BUG: 416140


(cherry picked from commit 5dd8cc91)
parent 0f65d894
Pipeline #234842 passed with stage
in 18 minutes and 27 seconds
/*
SPDX-FileCopyrightText: 2020 Benjamin Port <benjamin.port@enioka.com>
SPDX-FileCopyrightText: 2022 Harald Sitter <sitter@kde.org>
SPDX-License-Identifier: GPL-2.0-or-later
*/
......@@ -211,18 +212,18 @@ public:
// sub pixel
KXftConfig::SubPixel::Type spType = KXftConfig::SubPixel::NotSet;
xft.getSubPixelType(spType);
// if it is not set, we set it to rgb
// if it is not set, we have no subpixel hinting
if (spType == KXftConfig::SubPixel::NotSet) {
spType = KXftConfig::SubPixel::Rgb;
spType = KXftConfig::SubPixel::None;
}
setSubPixel(spType);
// hinting
KXftConfig::Hint::Style hStyle = KXftConfig::Hint::NotSet;
xft.getHintStyle(hStyle);
// if it is not set, we set it to slight hinting
// if it is not set, we have no hinting
if (hStyle == KXftConfig::Hint::NotSet) {
hStyle = KXftConfig::Hint::Slight;
hStyle = KXftConfig::Hint::None;
}
setHinting(hStyle);
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment