]> cloud.milkyroute.net Git - dolphin.git/commitdiff
ViewPropertiesDialog: disable Apply button on startup
authorElvis Angelaccio <elvis.angelaccio@kde.org>
Wed, 13 Jun 2018 21:16:33 +0000 (23:16 +0200)
committerElvis Angelaccio <elvis.angelaccio@kde.org>
Wed, 13 Jun 2018 21:16:33 +0000 (23:16 +0200)
We call `markAsDirty(false)` at the end of `loadSettings()` but that's
not enough, because `m_isDirty` is initialized to false (as it should be).
This means `isDirtyChanged` is not emitted and the button is not disabled.

By disabling it explicitly, we make it consistent with the Apply button
in the settings dialog (which is also initially disabled).

src/settings/viewpropertiesdialog.cpp

index dbbf5fa68336048d26cb63898eb20a45e6dd1101..60c9f21f56160659a8fd17a25c32d22713812ccf 100644 (file)
@@ -196,6 +196,7 @@ ViewPropertiesDialog::ViewPropertiesDialog(DolphinView* dolphinView) :
     okButton->setDefault(true);
 
     auto applyButton = buttonBox->button(QDialogButtonBox::Apply);
+    applyButton->setEnabled(false);
     connect(applyButton, &QPushButton::clicked, this, &ViewPropertiesDialog::slotApply);
     connect(this, &ViewPropertiesDialog::isDirtyChanged, applyButton, [applyButton](bool isDirty) {
         applyButton->setEnabled(isDirty);