From 72378fd36c92686e8a7d9dccb8df8c3727cc887d Mon Sep 17 00:00:00 2001
From: Karol Kosek <krkk@serenityos.org>
Date: Wed, 13 Oct 2021 22:09:38 +0200
Subject: [PATCH] FontEditor: Reuse the request_close() function in Open action

---
 Userland/Applications/FontEditor/FontEditor.cpp | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/Userland/Applications/FontEditor/FontEditor.cpp b/Userland/Applications/FontEditor/FontEditor.cpp
index f4e8b4b9c2..e17adbc5ca 100644
--- a/Userland/Applications/FontEditor/FontEditor.cpp
+++ b/Userland/Applications/FontEditor/FontEditor.cpp
@@ -165,16 +165,9 @@ FontEditorWidget::FontEditorWidget(const String& path, RefPtr<Gfx::BitmapFont>&&
     });
     m_new_action->set_status_tip("Create a new font");
     m_open_action = GUI::CommonActions::make_open_action([&](auto&) {
-        if (window()->is_modified()) {
-            auto result = GUI::MessageBox::show(window(), "Save changes to the current font?", "Unsaved changes", GUI::MessageBox::Type::Warning, GUI::MessageBox::InputType::YesNoCancel);
-            if (result == GUI::Dialog::ExecResult::ExecYes) {
-                m_save_action->activate();
-                if (window()->is_modified())
-                    return;
-            }
-            if (result == GUI::Dialog::ExecResult::ExecCancel)
-                return;
-        }
+        if (!request_close())
+            return;
+
         Optional<String> open_path = GUI::FilePicker::get_open_filepath(window(), {}, "/res/fonts/");
         if (!open_path.has_value())
             return;