From 0264d3de4574f8e397ef6b321f9084a4ef83fdb7 Mon Sep 17 00:00:00 2001 From: Karol Kosek Date: Sat, 17 Jul 2021 16:50:57 +0200 Subject: [PATCH] HackStudio: Remove noop when deciding whether to disable delete action The iterator in has_permissions will just be equal to sections.end() when there are no selected files. --- Userland/DevTools/HackStudio/HackStudioWidget.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Userland/DevTools/HackStudio/HackStudioWidget.cpp b/Userland/DevTools/HackStudio/HackStudioWidget.cpp index d84d0db7c2..52edaa2881 100644 --- a/Userland/DevTools/HackStudio/HackStudioWidget.cpp +++ b/Userland/DevTools/HackStudio/HackStudioWidget.cpp @@ -983,7 +983,7 @@ void HackStudioWidget::configure_project_tree_view() return access(m_project->model().full_path(selected_file.parent()).characters(), W_OK) == 0; }); bool has_permissions = it != selections.end(); - m_delete_action->set_enabled(!m_project_tree_view->selection().is_empty() && has_permissions); + m_delete_action->set_enabled(has_permissions); }; m_project_tree_view->on_activation = [this](auto& index) {