DisplaySettings: Revert changes unless confirmed

1 files changed, 7 insertions, 1 deletions

diff --git a/Userland/Applications/DisplaySettings/MonitorSettingsWidget.cpp b/Userland/Applications/DisplaySettings/MonitorSettingsWidget.cpp
index abff8e00e1..e733b51167 100644
--- a/Userland/Applications/DisplaySettings/MonitorSettingsWidget.cpp
+++ b/Userland/Applications/DisplaySettings/MonitorSettingsWidget.cpp
@@ -121,12 +121,18 @@ void MonitorSettingsWidget::apply_settings()
             });
 
             // If the user selects "No", closes the window or the window gets closed by the 10 seconds timer, revert the changes.
-            if (box->exec() != GUI::MessageBox::ExecYes) {
+            if (box->exec() == GUI::MessageBox::ExecYes) {
                 auto save_result = GUI::WindowServerConnection::the().save_screen_layout();
                 if (!save_result.success()) {
                     GUI::MessageBox::show(window(), String::formatted("Error saving settings: {}", save_result.error_msg()),
                         "Unable to save setting", GUI::MessageBox::Type::Error);
                 }
+            } else {
+                auto restore_result = GUI::WindowServerConnection::the().set_screen_layout(current_layout, false);
+                if (!restore_result.success()) {
+                    GUI::MessageBox::show(window(), String::formatted("Error restoring settings: {}", restore_result.error_msg()),
+                        "Unable to restore setting", GUI::MessageBox::Type::Error);
+                }
             }
         } else {
             GUI::MessageBox::show(window(), String::formatted("Error setting screen layout: {}", result.error_msg()),