From 5157a6f6b3a70a168f0a3bd08c9d9baa8f8f2051 Mon Sep 17 00:00:00 2001 From: Karol Kosek Date: Tue, 10 Jan 2023 22:48:33 +0100 Subject: [PATCH] ThemeEditor: Fully load files on drop events Previously, only the preview frame was updated without changing any values in the right panel or even a file path in the window title. --- Userland/Applications/ThemeEditor/MainWidget.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Userland/Applications/ThemeEditor/MainWidget.cpp b/Userland/Applications/ThemeEditor/MainWidget.cpp index d88dfd9a1f..6c123a04d8 100644 --- a/Userland/Applications/ThemeEditor/MainWidget.cpp +++ b/Userland/Applications/ThemeEditor/MainWidget.cpp @@ -667,13 +667,13 @@ void MainWidget::drop_event(GUI::DropEvent& event) return; } - auto response = FileSystemAccessClient::Client::the().try_request_file_deprecated(window(), urls.first().path(), Core::OpenMode::ReadOnly); + auto response = FileSystemAccessClient::Client::the().request_file(window(), urls.first().path(), Core::Stream::OpenMode::Read); if (response.is_error()) return; - auto set_theme_from_file_result = m_preview_widget->set_theme_from_file(response.release_value()); - if (set_theme_from_file_result.is_error()) - GUI::MessageBox::show_error(window(), DeprecatedString::formatted("Setting theme from file has failed: {}", set_theme_from_file_result.error())); + auto load_from_file_result = load_from_file(response.value().filename(), response.value().release_stream()); + if (load_from_file_result.is_error()) + GUI::MessageBox::show_error(window(), DeprecatedString::formatted("Can't open file named {}: {}", response.value().filename(), load_from_file_result.error())); } }