diff --git a/editor/directory_create_dialog.cpp b/editor/directory_create_dialog.cpp new file mode 100644 index 000000000000..df860bab2c00 --- /dev/null +++ b/editor/directory_create_dialog.cpp @@ -0,0 +1,172 @@ +/**************************************************************************/ +/* directory_create_dialog.cpp */ +/**************************************************************************/ +/* This file is part of: */ +/* GODOT ENGINE */ +/* https://godotengine.org */ +/**************************************************************************/ +/* Copyright (c) 2014-present Godot Engine contributors (see AUTHORS.md). */ +/* Copyright (c) 2007-2014 Juan Linietsky, Ariel Manzur. */ +/* */ +/* Permission is hereby granted, free of charge, to any person obtaining */ +/* a copy of this software and associated documentation files (the */ +/* "Software"), to deal in the Software without restriction, including */ +/* without limitation the rights to use, copy, modify, merge, publish, */ +/* distribute, sublicense, and/or sell copies of the Software, and to */ +/* permit persons to whom the Software is furnished to do so, subject to */ +/* the following conditions: */ +/* */ +/* The above copyright notice and this permission notice shall be */ +/* included in all copies or substantial portions of the Software. */ +/* */ +/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */ +/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */ +/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. */ +/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */ +/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */ +/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */ +/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */ +/**************************************************************************/ + +#include "directory_create_dialog.h" + +#include "core/io/dir_access.h" +#include "editor/editor_node.h" +#include "editor/editor_scale.h" +#include "scene/gui/box_container.h" +#include "scene/gui/label.h" +#include "scene/gui/line_edit.h" +#include "scene/gui/panel_container.h" + +static String sanitize_input(const String &p_path) { + String path = p_path.strip_edges(); + if (path.ends_with("/")) { + path = path.left(path.length() - 1); + } + return path; +} + +String DirectoryCreateDialog::_validate_path(const String &p_path) const { + if (p_path.is_empty()) { + return TTR("Folder name cannot be empty."); + } + + for (const String &part : p_path.split("/")) { + if (part.is_empty()) { + return TTR("Folder name cannot be empty."); + } + if (p_path.contains("\\") || p_path.contains(":") || p_path.contains("*") || + p_path.contains("|") || p_path.contains(">") || p_path.ends_with(".") || p_path.ends_with(" ")) { + return TTR("Folder name contains invalid characters."); + } + } + + Ref da = DirAccess::create(DirAccess::ACCESS_RESOURCES); + da->change_dir(base_dir); + if (da->file_exists(p_path)) { + return TTR("File with that name already exists."); + } + if (da->dir_exists(p_path)) { + return TTR("Folder with that name already exists."); + } + + return String(); +} + +void DirectoryCreateDialog::_on_dir_path_changed(const String &p_text) { + const String path = sanitize_input(p_text); + const String error = _validate_path(path); + + if (error.is_empty()) { + status_label->add_theme_color_override("font_color", get_theme_color(SNAME("success_color"), SNAME("Editor"))); + + if (path.contains("/")) { + status_label->set_text(TTR("Using slashes in folder names will create subfolders recursively.")); + } else { + status_label->set_text(TTR("Folder name is valid.")); + } + } else { + status_label->add_theme_color_override("font_color", get_theme_color(SNAME("error_color"), SNAME("Editor"))); + status_label->set_text(error); + } + + get_ok_button()->set_disabled(!error.is_empty()); +} + +void DirectoryCreateDialog::ok_pressed() { + const String path = sanitize_input(dir_path->get_text()); + + // The OK button should be disabled if the path is invalid, but just in case. + const String error = _validate_path(path); + ERR_FAIL_COND_MSG(!error.is_empty(), error); + + Error err; + Ref da = DirAccess::create(DirAccess::ACCESS_RESOURCES); + + err = da->change_dir(base_dir); + ERR_FAIL_COND_MSG(err != OK, "Cannot open directory '" + base_dir + "'."); + + print_verbose("Making folder " + path + " in " + base_dir); + err = da->make_dir_recursive(path); + + if (err == OK) { + emit_signal(SNAME("dir_created")); + } else { + EditorNode::get_singleton()->show_warning(TTR("Could not create folder.")); + } + hide(); +} + +void DirectoryCreateDialog::_post_popup() { + ConfirmationDialog::_post_popup(); + dir_path->grab_focus(); +} + +void DirectoryCreateDialog::config(const String &p_base_dir) { + base_dir = p_base_dir; + label->set_text(vformat(TTR("Create new folder in %s:"), base_dir)); + dir_path->set_text("new folder"); + dir_path->select_all(); + _on_dir_path_changed(dir_path->get_text()); +} + +void DirectoryCreateDialog::_bind_methods() { + ADD_SIGNAL(MethodInfo("dir_created")); +} + +void DirectoryCreateDialog::_notification(int p_what) { + switch (p_what) { + case NOTIFICATION_THEME_CHANGED: { + status_panel->add_theme_style_override("panel", get_theme_stylebox(SNAME("panel"), SNAME("Tree"))); + } break; + } +} + +DirectoryCreateDialog::DirectoryCreateDialog() { + set_title(TTR("Create Folder")); + set_min_size(Size2i(480, 0) * EDSCALE); + + VBoxContainer *vb = memnew(VBoxContainer); + add_child(vb); + + label = memnew(Label); + label->set_text_overrun_behavior(TextServer::OVERRUN_TRIM_WORD_ELLIPSIS); + vb->add_child(label); + + dir_path = memnew(LineEdit); + dir_path->connect("text_changed", callable_mp(this, &DirectoryCreateDialog::_on_dir_path_changed)); + vb->add_child(dir_path); + register_text_enter(dir_path); + + Control *spacing = memnew(Control); + spacing->set_custom_minimum_size(Size2(0, 10 * EDSCALE)); + vb->add_child(spacing); + + status_panel = memnew(PanelContainer); + status_panel->set_v_size_flags(Control::SIZE_EXPAND_FILL); + vb->add_child(status_panel); + + status_label = memnew(Label); + status_label->set_v_size_flags(Control::SIZE_EXPAND_FILL); + status_panel->add_child(status_label); +} diff --git a/editor/directory_create_dialog.h b/editor/directory_create_dialog.h new file mode 100644 index 000000000000..e2601181da81 --- /dev/null +++ b/editor/directory_create_dialog.h @@ -0,0 +1,68 @@ +/**************************************************************************/ +/* directory_create_dialog.h */ +/**************************************************************************/ +/* This file is part of: */ +/* GODOT ENGINE */ +/* https://godotengine.org */ +/**************************************************************************/ +/* Copyright (c) 2014-present Godot Engine contributors (see AUTHORS.md). */ +/* Copyright (c) 2007-2014 Juan Linietsky, Ariel Manzur. */ +/* */ +/* Permission is hereby granted, free of charge, to any person obtaining */ +/* a copy of this software and associated documentation files (the */ +/* "Software"), to deal in the Software without restriction, including */ +/* without limitation the rights to use, copy, modify, merge, publish, */ +/* distribute, sublicense, and/or sell copies of the Software, and to */ +/* permit persons to whom the Software is furnished to do so, subject to */ +/* the following conditions: */ +/* */ +/* The above copyright notice and this permission notice shall be */ +/* included in all copies or substantial portions of the Software. */ +/* */ +/* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, */ +/* EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF */ +/* MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. */ +/* IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY */ +/* CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, */ +/* TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE */ +/* SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. */ +/**************************************************************************/ + +#ifndef DIRECTORY_CREATE_DIALOG_H +#define DIRECTORY_CREATE_DIALOG_H + +#include "scene/gui/dialogs.h" + +class Label; +class LineEdit; +class PanelContainer; + +class DirectoryCreateDialog : public ConfirmationDialog { + GDCLASS(DirectoryCreateDialog, ConfirmationDialog); + + String base_dir; + + Label *label = nullptr; + LineEdit *dir_path = nullptr; + + PanelContainer *status_panel = nullptr; + Label *status_label = nullptr; + + String _validate_path(const String &p_path) const; + + void _on_dir_path_changed(const String &p_text); + +protected: + static void _bind_methods(); + void _notification(int p_what); + + virtual void ok_pressed() override; + virtual void _post_popup() override; + +public: + void config(const String &p_base_dir); + + DirectoryCreateDialog(); +}; + +#endif // DIRECTORY_CREATE_DIALOG_H diff --git a/editor/filesystem_dock.cpp b/editor/filesystem_dock.cpp index 066e8cb84ef1..4a76ac039b1a 100644 --- a/editor/filesystem_dock.cpp +++ b/editor/filesystem_dock.cpp @@ -38,6 +38,7 @@ #include "core/os/os.h" #include "core/templates/list.h" #include "editor/create_dialog.h" +#include "editor/directory_create_dialog.h" #include "editor/editor_feature_profile.h" #include "editor/editor_node.h" #include "editor/editor_resource_preview.h" @@ -1514,43 +1515,6 @@ void FileSystemDock::_save_scenes_after_move(const HashMap &p_re EditorNode::get_singleton()->save_scene_list(new_filenames); } -void FileSystemDock::_make_dir_confirm() { - String dir_name = make_dir_dialog_text->get_text().strip_edges(); - - if (dir_name.length() == 0) { - EditorNode::get_singleton()->show_warning(TTR("No name provided.")); - return; - } else if (dir_name.contains("/") || dir_name.contains("\\") || dir_name.contains(":") || dir_name.contains("*") || - dir_name.contains("|") || dir_name.contains(">") || dir_name.ends_with(".") || dir_name.ends_with(" ")) { - EditorNode::get_singleton()->show_warning(TTR("Provided name contains invalid characters.")); - return; - } - - String directory = path; - if (!directory.ends_with("/")) { - directory = directory.get_base_dir(); - } - - print_verbose("Making folder " + dir_name + " in " + directory); - Ref da = DirAccess::create(DirAccess::ACCESS_RESOURCES); - Error err = da->change_dir(directory); - ERR_FAIL_COND_MSG(err != OK, "Cannot open directory '" + directory + "'."); - - if (da->dir_exists(dir_name)) { - EditorNode::get_singleton()->show_warning(TTR("Could not create folder. File with that name already exists.")); - return; - } - - err = da->make_dir(dir_name); - - if (err == OK) { - print_verbose("FileSystem: calling rescan."); - _rescan(); - } else { - EditorNode::get_singleton()->show_warning(TTR("Could not create folder.")); - } -} - void FileSystemDock::_make_scene_confirm() { const String scene_path = make_scene_dialog->get_scene_path(); @@ -2108,10 +2072,12 @@ void FileSystemDock::_file_option(int p_option, const Vector &p_selected } break; case FILE_NEW_FOLDER: { - make_dir_dialog_text->set_text("new folder"); - make_dir_dialog_text->select_all(); - make_dir_dialog->popup_centered(Size2(250, 80) * EDSCALE); - make_dir_dialog_text->grab_focus(); + String directory = path; + if (!directory.ends_with("/")) { + directory = directory.get_base_dir(); + } + make_dir_dialog->config(directory); + make_dir_dialog->popup_centered(); } break; case FILE_NEW_SCENE: { @@ -3352,16 +3318,9 @@ FileSystemDock::FileSystemDock() { duplicate_dialog->register_text_enter(duplicate_dialog_text); duplicate_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_duplicate_operation_confirm)); - make_dir_dialog = memnew(ConfirmationDialog); - make_dir_dialog->set_title(TTR("Create Folder")); - VBoxContainer *make_folder_dialog_vb = memnew(VBoxContainer); - make_dir_dialog->add_child(make_folder_dialog_vb); - - make_dir_dialog_text = memnew(LineEdit); - make_folder_dialog_vb->add_margin_child(TTR("Name:"), make_dir_dialog_text); + make_dir_dialog = memnew(DirectoryCreateDialog); add_child(make_dir_dialog); - make_dir_dialog->register_text_enter(make_dir_dialog_text); - make_dir_dialog->connect("confirmed", callable_mp(this, &FileSystemDock::_make_dir_confirm)); + make_dir_dialog->connect("dir_created", callable_mp(this, &FileSystemDock::_rescan)); make_scene_dialog = memnew(SceneCreateDialog); add_child(make_scene_dialog); diff --git a/editor/filesystem_dock.h b/editor/filesystem_dock.h index 7c67be44bea3..e47178d294df 100644 --- a/editor/filesystem_dock.h +++ b/editor/filesystem_dock.h @@ -49,6 +49,7 @@ class LineEdit; class ProgressBar; class SceneCreateDialog; class ShaderCreateDialog; +class DirectoryCreateDialog; class FileSystemDock : public VBoxContainer { GDCLASS(FileSystemDock, VBoxContainer); @@ -155,8 +156,7 @@ private: LineEdit *rename_dialog_text = nullptr; ConfirmationDialog *duplicate_dialog = nullptr; LineEdit *duplicate_dialog_text = nullptr; - ConfirmationDialog *make_dir_dialog = nullptr; - LineEdit *make_dir_dialog_text = nullptr; + DirectoryCreateDialog *make_dir_dialog = nullptr; ConfirmationDialog *overwrite_dialog = nullptr; SceneCreateDialog *make_scene_dialog = nullptr; ScriptCreateDialog *make_script_dialog = nullptr; @@ -241,7 +241,6 @@ private: void _folder_removed(String p_folder); void _resource_created(); - void _make_dir_confirm(); void _make_scene_confirm(); void _rename_operation_confirm(); void _duplicate_operation_confirm();