Commit eede3025 authored by Mikhail Zolotukhin's avatar Mikhail Zolotukhin

[Various KCMs] Notify about changes in GTK related settings

Summary:
Also save colors before saving colorScheme name, so that we can
correctly use heuristic in GTK Config, after the moment we know, that
color scheme has changed.

Test Plan: Check if the order of saving colors and saving color scheme name does not break anything.

Reviewers: #plasma, ngraham, ervin, bport, meven

Reviewed By: ngraham

Subscribers: ngraham, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27956
parent 491ce0f1
......@@ -343,10 +343,13 @@ void KCMColors::load()
void KCMColors::save()
{
ManagedConfigModule::save();
// We need to save the colors change first, to avoid a situation,
// when we announced that the color scheme has changed, but
// the colors themselves in the color scheme have not yet
if (m_selectedSchemeDirty) {
saveColors();
}
ManagedConfigModule::save();
m_activeSchemeEdited = false;
processPendingDeletions();
......
......@@ -4,3 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=colorScheme
......@@ -4,3 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=font
......@@ -4,3 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=Theme
......@@ -4,3 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=iconsOnButtons,iconsInMenus,toolButtonStyle
......@@ -4,4 +4,4 @@ Mutators=true
DefaultValueGetters=true
GenerateProperties=true
ParentInConstructor=true
Notifiers=animationDurationFactor
Notifiers=scrollbarLeftClickNavigatesByPage,animationDurationFactor
Markdown is supported
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