Skip to content

Commit

Permalink
QMessage should not be used to notify that. Here we should use the re…
Browse files Browse the repository at this point in the history
…quireRestart functionality.
  • Loading branch information
PaddleStroke authored and chennes committed Jul 31, 2023
1 parent 5cd6da6 commit 92b2635
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions src/Gui/DlgSettingsNotificationArea.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -41,18 +41,12 @@ DlgSettingsNotificationArea::DlgSettingsNotificationArea(QWidget* parent)
ui->setupUi(this);

adaptUiToAreaEnabledState(ui->NotificationAreaEnabled->isChecked());

connect(ui->NotificationAreaEnabled, &QCheckBox::stateChanged, [this](int state) {
bool enabled = state == Qt::CheckState::Checked;
this->adaptUiToAreaEnabledState(enabled);

if (enabled) {
QMessageBox::information(this,
tr("Notification Area"),
tr("Activation of the Notification Area only takes effect "
"after an application restart."));
// N.B: Deactivation is handled by the Notification Area itself, as it listens to all
// its configuration parameters.
this->requireRestart();
}
});
}
Expand Down

0 comments on commit 92b2635

Please sign in to comment.