From acea9036c17011b8af03a1c4c1905a5eec1cc454 Mon Sep 17 00:00:00 2001 From: Tanbir Jishan Date: Sat, 27 Aug 2022 21:24:37 +0530 Subject: [PATCH] kcms/colors: handle the "transparent" value explicitly --- kcms/colors/colorsapplicator.cpp | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/kcms/colors/colorsapplicator.cpp b/kcms/colors/colorsapplicator.cpp index 385b818da..b6413a81a 100644 --- a/kcms/colors/colorsapplicator.cpp +++ b/kcms/colors/colorsapplicator.cpp @@ -126,13 +126,17 @@ void applyScheme(const QString &colorSchemePath, KConfig *configOutput, KConfig: KSharedConfigPtr globalConfig = KSharedConfig::openConfig(QStringLiteral("kdeglobals")); globalConfig->sync(); - const auto hasAccent = [globalConfig]() { - return globalConfig->group("General").hasKey("AccentColor"); - }; const auto getAccent = [globalConfig]() { return globalConfig->group("General").readEntry("AccentColor", QColor()); }; + const auto hasAccent = [globalConfig, &getAccent]() { + if (getAccent() == QColor(Qt::transparent)) { + return false; + } + return globalConfig->group("General").hasKey("AccentColor"); + }; + // Using KConfig::SimpleConfig because otherwise Header colors won't be // rewritten when a new color scheme is loaded. KSharedConfigPtr config = KSharedConfig::openConfig(colorSchemePath, KConfig::SimpleConfig);