From 0083fe0118a4816ac05a6c10e4f71469c0d40ff3 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Mon, 17 Aug 2020 19:52:01 +0200 Subject: [PATCH] FileManager: Use DirectoryView for both windowed and desktop mode This patch begins the process of converting FileManager's desktop mode to using the DirectoryView class. This will allow us to share most of the code between the two different runtime modes of FileManager. We lose a bit of functionality in this patch, but we'll bring it back! --- Applications/FileManager/DirectoryView.cpp | 161 +++++++++++++++------ Applications/FileManager/DirectoryView.h | 34 ++++- Applications/FileManager/main.cpp | 116 ++------------- 3 files changed, 163 insertions(+), 148 deletions(-) diff --git a/Applications/FileManager/DirectoryView.cpp b/Applications/FileManager/DirectoryView.cpp index 0c5288a927..3c25ba7afc 100644 --- a/Applications/FileManager/DirectoryView.cpp +++ b/Applications/FileManager/DirectoryView.cpp @@ -27,8 +27,11 @@ #include "DirectoryView.h" #include #include +#include #include #include +#include +#include #include #include @@ -97,6 +100,10 @@ void DirectoryView::handle_activation(const GUI::ModelIndex& index) } if (S_ISDIR(st.st_mode)) { + if (is_desktop()) { + Desktop::Launcher::open(URL::create_with_file_protocol(path)); + return; + } open(path); return; } @@ -104,33 +111,36 @@ void DirectoryView::handle_activation(const GUI::ModelIndex& index) auto url = URL::create_with_file_protocol(path); auto launcher_handlers = get_launch_handlers(url); auto default_launcher = get_default_launch_handler(launcher_handlers); - if (default_launcher && on_launch) { - on_launch(url, *default_launcher); + if (default_launcher) { + launch(url, *default_launcher); } else { auto error_message = String::format("Could not open %s", path.characters()); GUI::MessageBox::show(window(), error_message, "File Manager", GUI::MessageBox::Type::Error); } } -DirectoryView::DirectoryView() - : m_model(GUI::FileSystemModel::create()) +DirectoryView::DirectoryView(Mode mode) + : m_mode(mode) + , m_model(GUI::FileSystemModel::create()) , m_sorting_model(GUI::SortingProxyModel::create(m_model)) { set_active_widget(nullptr); set_content_margins({ 2, 2, 2, 2 }); - m_icon_view = add(); - m_icon_view->set_model(m_sorting_model); - m_columns_view = add(); - m_columns_view->set_model(m_sorting_model); + setup_model(); - m_table_view = add(); - m_table_view->set_model(m_sorting_model); + setup_icon_view(); + if (mode != Mode::Desktop) { + setup_columns_view(); + setup_table_view(); + } - m_table_view->set_key_column_and_sort_order(GUI::FileSystemModel::Column::Name, GUI::SortOrder::Ascending); + set_view_mode(ViewMode::Icon); +} - m_icon_view->set_model_column(GUI::FileSystemModel::Column::Name); - m_columns_view->set_model_column(GUI::FileSystemModel::Column::Name); +void DirectoryView::setup_model() +{ + m_model->set_root_path(Core::StandardPaths::desktop_directory()); m_model->on_error = [this](int error, const char* error_string) { bool quit = false; @@ -144,8 +154,10 @@ DirectoryView::DirectoryView() }; m_model->on_complete = [this] { - m_table_view->selection().clear(); - m_icon_view->selection().clear(); + if (m_table_view) + m_table_view->selection().clear(); + if (m_icon_view) + m_icon_view->selection().clear(); add_path_to_history(model().root_path()); @@ -159,13 +171,72 @@ DirectoryView::DirectoryView() if (on_thumbnail_progress) on_thumbnail_progress(done, total); }; +} +void DirectoryView::setup_icon_view() +{ + m_icon_view = add(); + + if (is_desktop()) { + m_icon_view->set_frame_shape(Gfx::FrameShape::NoFrame); + m_icon_view->set_scrollbars_enabled(false); + m_icon_view->set_fill_with_background_color(false); + } + + m_icon_view->set_model(m_sorting_model); + m_icon_view->set_model_column(GUI::FileSystemModel::Column::Name); m_icon_view->on_activation = [&](auto& index) { handle_activation(map_index(index)); }; + m_icon_view->on_selection_change = [this] { + update_statusbar(); + if (on_selection_change) + on_selection_change(*m_icon_view); + }; + m_icon_view->on_context_menu_request = [this](auto& index, auto& event) { + if (on_context_menu_request) + on_context_menu_request(*m_icon_view, map_index(index), event); + }; + m_icon_view->on_drop = [this](auto& index, auto& event) { + if (on_drop) + on_drop(*m_icon_view, map_index(index), event); + }; +} + +void DirectoryView::setup_columns_view() +{ + m_columns_view = add(); + m_columns_view->set_model(m_sorting_model); + m_columns_view->set_model_column(GUI::FileSystemModel::Column::Name); + m_columns_view->on_activation = [&](auto& index) { handle_activation(map_index(index)); }; + + m_columns_view->on_selection_change = [this] { + update_statusbar(); + if (on_selection_change) + on_selection_change(*m_columns_view); + }; + + m_columns_view->on_context_menu_request = [this](auto& index, auto& event) { + if (on_context_menu_request) + on_context_menu_request(*m_columns_view, map_index(index), event); + }; + + m_columns_view->on_drop = [this](auto& index, auto& event) { + if (on_drop) + on_drop(*m_columns_view, map_index(index), event); + }; +} + +void DirectoryView::setup_table_view() +{ + m_table_view = add(); + m_table_view->set_model(m_sorting_model); + + m_table_view->set_key_column_and_sort_order(GUI::FileSystemModel::Column::Name, GUI::SortOrder::Ascending); + m_table_view->on_activation = [&](auto& index) { handle_activation(map_index(index)); }; @@ -175,44 +246,16 @@ DirectoryView::DirectoryView() if (on_selection_change) on_selection_change(*m_table_view); }; - m_icon_view->on_selection_change = [this] { - update_statusbar(); - if (on_selection_change) - on_selection_change(*m_icon_view); - }; - m_columns_view->on_selection_change = [this] { - update_statusbar(); - if (on_selection_change) - on_selection_change(*m_columns_view); - }; m_table_view->on_context_menu_request = [this](auto& index, auto& event) { if (on_context_menu_request) on_context_menu_request(*m_table_view, map_index(index), event); }; - m_icon_view->on_context_menu_request = [this](auto& index, auto& event) { - if (on_context_menu_request) - on_context_menu_request(*m_icon_view, map_index(index), event); - }; - m_columns_view->on_context_menu_request = [this](auto& index, auto& event) { - if (on_context_menu_request) - on_context_menu_request(*m_columns_view, map_index(index), event); - }; m_table_view->on_drop = [this](auto& index, auto& event) { if (on_drop) on_drop(*m_table_view, map_index(index), event); }; - m_icon_view->on_drop = [this](auto& index, auto& event) { - if (on_drop) - on_drop(*m_icon_view, map_index(index), event); - }; - m_columns_view->on_drop = [this](auto& index, auto& event) { - if (on_drop) - on_drop(*m_columns_view, map_index(index), event); - }; - - set_view_mode(ViewMode::Icon); } DirectoryView::~DirectoryView() @@ -354,3 +397,35 @@ void DirectoryView::set_should_show_dotfiles(bool show_dotfiles) { m_model->set_should_show_dotfiles(show_dotfiles); } + +void DirectoryView::launch(const URL&, const LauncherHandler& launcher_handler) +{ + pid_t child; + if (launcher_handler.details().launcher_type == Desktop::Launcher::LauncherType::Application) { + const char* argv[] = { launcher_handler.details().name.characters(), nullptr }; + posix_spawn(&child, launcher_handler.details().executable.characters(), nullptr, nullptr, const_cast(argv), environ); + if (disown(child) < 0) + perror("disown"); + } else { + for (auto& path : selected_file_paths()) { + const char* argv[] = { launcher_handler.details().name.characters(), path.characters(), nullptr }; + posix_spawn(&child, launcher_handler.details().executable.characters(), nullptr, nullptr, const_cast(argv), environ); + if (disown(child) < 0) + perror("disown"); + } + } +} + +Vector DirectoryView::selected_file_paths() const +{ + Vector paths; + auto& view = current_view(); + auto& model = *view.model(); + view.selection().for_each_index([&](const GUI::ModelIndex& index) { + auto parent_index = model.parent_index(index); + auto name_index = model.index(index.row(), GUI::FileSystemModel::Column::Name, parent_index); + auto path = name_index.data(GUI::ModelRole::Custom).to_string(); + paths.append(path); + }); + return paths; +} diff --git a/Applications/FileManager/DirectoryView.h b/Applications/FileManager/DirectoryView.h index a66ab237f9..528b4b5891 100644 --- a/Applications/FileManager/DirectoryView.h +++ b/Applications/FileManager/DirectoryView.h @@ -57,6 +57,11 @@ class DirectoryView final C_OBJECT(DirectoryView); public: + enum class Mode { + Desktop, + Normal, + }; + virtual ~DirectoryView() override; void open(const StringView& path); @@ -72,7 +77,8 @@ public: void refresh(); - Function on_launch; + void launch(const AK::URL&, const LauncherHandler&); + Function on_path_change; Function on_selection_change; Function on_context_menu_request; @@ -104,31 +110,49 @@ public: } } + const GUI::AbstractView& current_view() const + { + return const_cast(this)->current_view(); + } + template void for_each_view_implementation(Callback callback) { - callback(*m_table_view); - callback(*m_icon_view); - callback(*m_columns_view); + if (m_icon_view) + callback(*m_icon_view); + if (m_table_view) + callback(*m_table_view); + if (m_columns_view) + callback(*m_columns_view); } void set_should_show_dotfiles(bool); GUI::FileSystemModel& model() { return *m_model; } + bool is_desktop() const { return m_mode == Mode::Desktop; } + + Vector selected_file_paths() const; + private: - DirectoryView(); + explicit DirectoryView(Mode); const GUI::FileSystemModel& model() const { return *m_model; } // ^GUI::ModelClient virtual void model_did_update(unsigned) override; + void setup_model(); + void setup_icon_view(); + void setup_columns_view(); + void setup_table_view(); + void handle_activation(const GUI::ModelIndex&); GUI::ModelIndex map_index(const GUI::ModelIndex&) const; void set_status_message(const StringView&); void update_statusbar(); + Mode m_mode { Mode::Normal }; ViewMode m_view_mode { Invalid }; NonnullRefPtr m_model; diff --git a/Applications/FileManager/main.cpp b/Applications/FileManager/main.cpp index 11becb159c..488a3c6d3a 100644 --- a/Applications/FileManager/main.cpp +++ b/Applications/FileManager/main.cpp @@ -66,7 +66,7 @@ #include #include -static int run_in_desktop_mode(RefPtr, String initial_location); +static int run_in_desktop_mode(RefPtr); static int run_in_windowed_mode(RefPtr, String initial_location); static Gfx::Bitmap& folder_icon() @@ -108,7 +108,7 @@ int main(int argc, char** argv) } if (app->args().contains_slow("--desktop") || app->args().contains_slow("-d")) - return run_in_desktop_mode(move(config), Core::StandardPaths::desktop_directory()); + return run_in_desktop_mode(move(config)); // our initial location is defined as, in order of precedence: // 1. the first command-line argument (e.g. FileManager /bin) @@ -147,14 +147,13 @@ private: } }; -int run_in_desktop_mode(RefPtr config, String initial_location) +int run_in_desktop_mode(RefPtr config) { static constexpr const char* process_name = "FileManager (Desktop)"; set_process_name(process_name, strlen(process_name)); pthread_setname_np(pthread_self(), process_name); (void)config; - (void)initial_location; auto window = GUI::Window::construct(); window->set_title("Desktop Manager"); window->set_window_type(GUI::WindowType::Desktop); @@ -163,66 +162,12 @@ int run_in_desktop_mode(RefPtr config, String initial_location auto& desktop_widget = window->set_main_widget(); desktop_widget.set_layout(); - auto& icon_view = desktop_widget.add(); - icon_view.set_frame_thickness(0); - icon_view.set_scrollbars_enabled(false); - icon_view.set_fill_with_background_color(false); - - auto model = GUI::FileSystemModel::create(initial_location); - icon_view.set_model(model); - icon_view.set_model_column(GUI::FileSystemModel::Column::Name); - - icon_view.on_activation = [&](auto& index) { - if (!index.is_valid()) - return; - auto& node = model->node(index); - auto path = node.full_path(model); - Desktop::Launcher::open(URL::create_with_file_protocol(path)); - }; + auto& directory_view = desktop_widget.add(DirectoryView::Mode::Desktop); + (void)directory_view; auto desktop_view_context_menu = GUI::Menu::construct("Directory View"); - auto mkdir_action = GUI::Action::create("New directory...", {}, Gfx::Bitmap::load_from_file("/res/icons/16x16/mkdir.png"), [&](const GUI::Action&) { - String value; - if (GUI::InputBox::show(value, window, "Enter name:", "New directory") == GUI::InputBox::ExecOK && !value.is_empty()) { - auto new_dir_path = LexicalPath::canonicalized_path( - String::format("%s/%s", - model->root_path().characters(), - value.characters())); - int rc = mkdir(new_dir_path.characters(), 0777); - if (rc < 0) { - GUI::MessageBox::show(window, String::format("mkdir(\"%s\") failed: %s", new_dir_path.characters(), strerror(errno)), "Error", GUI::MessageBox::Type::Error); - } - } - }); - - auto touch_action = GUI::Action::create("New file...", {}, Gfx::Bitmap::load_from_file("/res/icons/16x16/new.png"), [&](const GUI::Action&) { - String value; - if (GUI::InputBox::show(value, window, "Enter name:", "New file") == GUI::InputBox::ExecOK && !value.is_empty()) { - auto new_file_path = LexicalPath::canonicalized_path( - String::format("%s/%s", - model->root_path().characters(), - value.characters())); - struct stat st; - int rc = stat(new_file_path.characters(), &st); - if ((rc < 0 && errno != ENOENT)) { - GUI::MessageBox::show(window, String::format("stat(\"%s\") failed: %s", new_file_path.characters(), strerror(errno)), "Error", GUI::MessageBox::Type::Error); - return; - } - if (rc == 0) { - GUI::MessageBox::show(window, String::format("%s: Already exists", new_file_path.characters()), "Error", GUI::MessageBox::Type::Error); - return; - } - int fd = creat(new_file_path.characters(), 0666); - if (fd < 0) { - GUI::MessageBox::show(window, String::format("creat(\"%s\") failed: %s", new_file_path.characters(), strerror(errno)), "Error", GUI::MessageBox::Type::Error); - return; - } - rc = close(fd); - assert(rc >= 0); - } - }); - +#if 0 auto file_manager_action = GUI::Action::create("Show in FileManager...", {}, Gfx::Bitmap::load_from_file("/res/icons/16x16/filetype-folder.png"), [&](const GUI::Action&) { Desktop::Launcher::open(URL::create_with_file_protocol(model->root_path())); }); @@ -238,10 +183,11 @@ int run_in_desktop_mode(RefPtr config, String initial_location desktop_view_context_menu->add_separator(); desktop_view_context_menu->add_action(display_properties_action); - icon_view.on_context_menu_request = [&](const GUI::ModelIndex& index, const GUI::ContextMenuEvent& event) { + directory_view.on_context_menu_request = [&](const GUI::ModelIndex& index, const GUI::ContextMenuEvent& event) { if (!index.is_valid()) desktop_view_context_menu->popup(event.screen_position()); }; +#endif auto wm_config = Core::ConfigFile::get_for_app("WindowManager"); auto selected_wallpaper = wm_config->read_entry("Background", "Wallpaper", ""); @@ -296,7 +242,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio tree_view.set_column_hidden(GUI::FileSystemModel::Column::SymlinkTarget, true); tree_view.set_size_policy(GUI::SizePolicy::Fixed, GUI::SizePolicy::Fill); tree_view.set_preferred_size(150, 0); - auto& directory_view = splitter.add(); + auto& directory_view = splitter.add(DirectoryView::Mode::Normal); auto& statusbar = widget.add(); @@ -438,19 +384,6 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio view_type_action_group->add_action(*view_as_table_action); view_type_action_group->add_action(*view_as_columns_action); - auto selected_file_paths = [&] { - Vector paths; - auto& view = directory_view.current_view(); - auto& model = *view.model(); - view.selection().for_each_index([&](const GUI::ModelIndex& index) { - auto parent_index = model.parent_index(index); - auto name_index = model.index(index.row(), GUI::FileSystemModel::Column::Name, parent_index); - auto path = name_index.data(GUI::ModelRole::Custom).to_string(); - paths.append(path); - }); - return paths; - }; - auto tree_view_selected_file_paths = [&] { Vector paths; auto& view = tree_view; @@ -465,8 +398,8 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio }); auto copy_action = GUI::CommonActions::make_copy_action( - [&](const GUI::Action&) { - Vector paths = selected_file_paths(); + [&](auto&) { + auto paths = directory_view.selected_file_paths(); if (!paths.size()) paths = tree_view_selected_file_paths(); @@ -493,7 +426,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio if (action.activator() == directory_context_menu || directory_view.active_widget()->is_focused()) { path = directory_view.path(); container_dir_path = path; - selected = selected_file_paths(); + selected = directory_view.selected_file_paths(); } else { path = directories_model->full_path(tree_view.selection().first()); container_dir_path = LexicalPath(path).basename(); @@ -530,7 +463,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio AK::String target_directory; if (action.activator() == directory_context_menu) - target_directory = selected_file_paths()[0]; + target_directory = directory_view.selected_file_paths()[0]; else target_directory = directory_view.path(); @@ -554,7 +487,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio }; auto do_delete = [&](ConfirmBeforeDelete confirm, const GUI::Action&) { - Vector paths = selected_file_paths(); + auto paths = directory_view.selected_file_paths(); if (!paths.size()) paths = tree_view_selected_file_paths(); @@ -814,23 +747,6 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio NonnullRefPtrVector current_file_handlers; RefPtr file_context_menu_action_default_action; - directory_view.on_launch = [&](const AK::URL&, const LauncherHandler& launcher_handler) { - pid_t child; - if (launcher_handler.details().launcher_type == Desktop::Launcher::LauncherType::Application) { - const char* argv[] = { launcher_handler.details().name.characters(), nullptr }; - posix_spawn(&child, launcher_handler.details().executable.characters(), nullptr, nullptr, const_cast(argv), environ); - if (disown(child) < 0) - perror("disown"); - } else { - for (auto& path : selected_file_paths()) { - const char* argv[] = { launcher_handler.details().name.characters(), path.characters(), nullptr }; - posix_spawn(&child, launcher_handler.details().executable.characters(), nullptr, nullptr, const_cast(argv), environ); - if (disown(child) < 0) - perror("disown"); - } - } - }; - directory_view.on_context_menu_request = [&](const GUI::AbstractView&, const GUI::ModelIndex& index, const GUI::ContextMenuEvent& event) { if (index.is_valid()) { auto& node = directory_view.model().node(index); @@ -853,7 +769,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio auto default_file_handler = directory_view.get_default_launch_handler(current_file_handlers); if (default_file_handler) { auto file_open_action = default_file_handler->create_launch_action([&, full_path = move(full_path)](auto& launcher_handler) { - directory_view.on_launch(URL::create_with_file_protocol(full_path), launcher_handler); + directory_view.launch(URL::create_with_file_protocol(full_path), launcher_handler); }); if (default_file_handler->details().launcher_type == Desktop::Launcher::LauncherType::Application) file_open_action->set_text(String::format("Run %s", file_open_action->text().characters())); @@ -875,7 +791,7 @@ int run_in_windowed_mode(RefPtr config, String initial_locatio if (&handler == default_file_handler.ptr()) continue; file_open_with_menu.add_action(handler.create_launch_action([&, full_path = move(full_path)](auto& launcher_handler) { - directory_view.on_launch(URL::create_with_file_protocol(full_path), launcher_handler); + directory_view.launch(URL::create_with_file_protocol(full_path), launcher_handler); })); } }