Commit 9a77503f authored by David Redondo's avatar David Redondo 🏎
Browse files

lookandfeelmanager: Write colors before color scheme

While we indeed write both with KConfig::Notify flag interested
processes might only listen to the scheme change instead of
every color scheme group. When they react to the scheme change
the colors have not yet changed.
The colors kcm also does it in this order.
BUG:421745
FIXED-IN:5.24.7
parent e0a51565
Pipeline #215921 passed with stage
in 11 minutes and 44 seconds
......@@ -190,12 +190,13 @@ void LookAndFeelManager::setColors(const QString &scheme, const QString &colorFi
KConfig configDefault(configDefaults(QStringLiteral("kdeglobals")));
auto kdeGlobalsCfg = KSharedConfig::openConfig(QStringLiteral("kdeglobals"), KConfig::FullConfig);
writeNewDefaults(*kdeGlobalsCfg, configDefault, QStringLiteral("General"), QStringLiteral("ColorScheme"), scheme, KConfig::Notify);
if (m_mode == Mode::Apply) {
applyScheme(colorFile, kdeGlobalsCfg.data(), KConfig::Notify);
}
writeNewDefaults(*kdeGlobalsCfg, configDefault, QStringLiteral("General"), QStringLiteral("ColorScheme"), scheme, KConfig::Notify);
Q_EMIT colorsChanged();
}
......
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