Qt: Remove duplicate gui repaints in settings dialog

ApplyGuiOptions(true) already forces GuiStylesheetRequest and therefore RepaintGui in main_window if necessary.
We don't need GuiRepaintRequest on accept because there should be no option left that doesn't repaint immediately
This commit is contained in:
Megamouse 2017-09-10 23:16:06 +02:00 committed by Ivan
parent f0358f85fc
commit fac8a520f0
1 changed files with 0 additions and 3 deletions

View File

@ -95,7 +95,6 @@ settings_dialog::settings_dialog(std::shared_ptr<gui_settings> xSettings, const
std::vector<std::string> selected_ls = std::vector<std::string>(selectedlle.begin(), selectedlle.end()); std::vector<std::string> selected_ls = std::vector<std::string>(selectedlle.begin(), selectedlle.end());
xemu_settings->SaveSelectedLibraries(selected_ls); xemu_settings->SaveSelectedLibraries(selected_ls);
xemu_settings->SaveSettings(); xemu_settings->SaveSettings();
Q_EMIT GuiRepaintRequest();
accept(); accept();
}); });
connect(ui->cancelButton, &QAbstractButton::clicked, this, &QWidget::close); connect(ui->cancelButton, &QAbstractButton::clicked, this, &QWidget::close);
@ -743,9 +742,7 @@ settings_dialog::settings_dialog(std::shared_ptr<gui_settings> xSettings, const
ApplyGuiOptions(true); ApplyGuiOptions(true);
xgui_settings->Reset(true); xgui_settings->Reset(true);
xgui_settings->ChangeToConfig(GUI::Default); xgui_settings->ChangeToConfig(GUI::Default);
Q_EMIT GuiStylesheetRequest(GUI::Default);
Q_EMIT GuiSettingsSyncRequest(); Q_EMIT GuiSettingsSyncRequest();
Q_EMIT GuiRepaintRequest();
AddConfigs(); AddConfigs();
AddStylesheets(); AddStylesheets();
AddColoredIcons(); AddColoredIcons();