From d8743a2872d33236558462dec2bf3a9984fa12f2 Mon Sep 17 00:00:00 2001 From: Matthew McNamara Date: Fri, 8 Dec 2017 01:32:40 +0100 Subject: [PATCH] ui: Fix unsaved changes dialog showing twice When clicking the close button with unsaved changes in a source properties dialog closeEvent and reject are called, which both call ConfirmQuit. This fixes it by setting the acceptClicked flag correctly in ConfirmQuit --- UI/window-basic-properties.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/UI/window-basic-properties.cpp b/UI/window-basic-properties.cpp index 1bd3a260e..d07336b26 100644 --- a/UI/window-basic-properties.cpp +++ b/UI/window-basic-properties.cpp @@ -288,6 +288,7 @@ bool OBSBasicProperties::ConfirmQuit() switch (button) { case QMessageBox::Save: + acceptClicked = true; if (view->DeferUpdate()) view->UpdateSettings(); // Do nothing because the settings are already updated