forked from qt-creator/qt-creator
Beautifier: Fix saving of settings
Applying didn't actually apply the new values. Fixes: QTCREATORBUG-29586 Change-Id: If9416cc18fffd269f6ca3d27523aa138e6f91d7a Reviewed-by: hjk <hjk@qt.io>
This commit is contained in:
@@ -221,6 +221,7 @@ public:
|
|||||||
|
|
||||||
setOnApply([&s, configurations] {
|
setOnApply([&s, configurations] {
|
||||||
s.customStyle.setValue(configurations->currentConfiguration());
|
s.customStyle.setValue(configurations->currentConfiguration());
|
||||||
|
settings().apply();
|
||||||
s.save();
|
s.save();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -82,9 +82,6 @@ public:
|
|||||||
fallbackStyle.addOption("WebKit");
|
fallbackStyle.addOption("WebKit");
|
||||||
fallbackStyle.setDefaultValue("Default");
|
fallbackStyle.setDefaultValue("Default");
|
||||||
|
|
||||||
predefinedStyle.setSettingsKey("predefinedStyle");
|
|
||||||
predefinedStyle.setDefaultValue("LLVM");
|
|
||||||
|
|
||||||
customStyle.setSettingsKey("customStyle");
|
customStyle.setSettingsKey("customStyle");
|
||||||
|
|
||||||
documentationFilePath = Core::ICore::userResourcePath(Constants::SETTINGS_DIRNAME)
|
documentationFilePath = Core::ICore::userResourcePath(Constants::SETTINGS_DIRNAME)
|
||||||
@@ -299,6 +296,7 @@ public:
|
|||||||
setOnApply([configurations, customizedStyleButton] {
|
setOnApply([configurations, customizedStyleButton] {
|
||||||
settings().usePredefinedStyle.setValue(!customizedStyleButton->isChecked());
|
settings().usePredefinedStyle.setValue(!customizedStyleButton->isChecked());
|
||||||
settings().customStyle.setValue(configurations->currentConfiguration());
|
settings().customStyle.setValue(configurations->currentConfiguration());
|
||||||
|
settings().apply();
|
||||||
settings().save();
|
settings().save();
|
||||||
});
|
});
|
||||||
|
|
||||||
|
|||||||
@@ -24,6 +24,7 @@ GeneralSettings &generalSettings()
|
|||||||
|
|
||||||
GeneralSettings::GeneralSettings()
|
GeneralSettings::GeneralSettings()
|
||||||
{
|
{
|
||||||
|
setAutoApply(false);
|
||||||
setSettingsGroups("Beautifier", "General");
|
setSettingsGroups("Beautifier", "General");
|
||||||
|
|
||||||
autoFormatOnSave.setSettingsKey(Utils::Constants::BEAUTIFIER_AUTO_FORMAT_ON_SAVE);
|
autoFormatOnSave.setSettingsKey(Utils::Constants::BEAUTIFIER_AUTO_FORMAT_ON_SAVE);
|
||||||
|
|||||||
@@ -222,6 +222,7 @@ public:
|
|||||||
|
|
||||||
setOnApply([&s, configurations] {
|
setOnApply([&s, configurations] {
|
||||||
s.customStyle.setValue(configurations->currentConfiguration());
|
s.customStyle.setValue(configurations->currentConfiguration());
|
||||||
|
settings().apply();
|
||||||
s.save();
|
s.save();
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user