godotengine/godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Changes for the "Recent Scripts" menu.

Browse Source
This commit is contained in:
Michael Alexsander Silva Dias 2018-01-07 22:31:36 -02:00
parent f13d034485
commit 140340978b
3 changed files with 50 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
editor/editor_settings.cpp
View File

@ -382,8 +382,6 @@ void EditorSettings::_load_defaults(Ref<ConfigFile> p_extra_config) {
_initial_set("text_editor/completion/callhint_tooltip_offset", Vector2()); _initial_set("text_editor/completion/callhint_tooltip_offset", Vector2());
_initial_set("text_editor/files/restore_scripts_on_load", true); _initial_set("text_editor/files/restore_scripts_on_load", true);
_initial_set("text_editor/completion/complete_file_paths", true); _initial_set("text_editor/completion/complete_file_paths", true);
_initial_set("text_editor/files/maximum_recent_files", 20);
hints["text_editor/files/maximum_recent_files"] = PropertyInfo(Variant::INT, "text_editor/files/maximum_recent_files", PROPERTY_HINT_RANGE, "1, 200, 1");
_initial_set("docks/scene_tree/start_create_dialog_fully_expanded", false); _initial_set("docks/scene_tree/start_create_dialog_fully_expanded", false);
_initial_set("docks/scene_tree/draw_relationship_lines", false); _initial_set("docks/scene_tree/draw_relationship_lines", false);

83
editor/plugins/script_editor_plugin.cpp
View File

@ -429,36 +429,32 @@ void ScriptEditor::_add_recent_script(String p_path) {
return; return;
} }
// remove if already stored Array rc = EditorSettings::get_singleton()->get_project_metadata("recent_files", "scripts", Array());
int already_recent = previous_scripts.find(p_path); if (rc.find(p_path) != -1) {
if (already_recent >= 0) { rc.erase(p_path);
previous_scripts.remove(already_recent); }
rc.push_front(p_path);
if (rc.size() > 10) {
rc.resize(10);
} }
// add to list EditorSettings::get_singleton()->set_project_metadata("recent_files", "scripts", rc);
previous_scripts.insert(0, p_path);
_update_recent_scripts(); _update_recent_scripts();
} }
void ScriptEditor::_update_recent_scripts() { void ScriptEditor::_update_recent_scripts() {
// make sure we don't exceed max size Array rc = EditorSettings::get_singleton()->get_project_metadata("recent_files", "scripts", Array());
const int max_history = EDITOR_DEF("text_editor/files/maximum_recent_files", 20);
if (previous_scripts.size() > max_history) {
previous_scripts.resize(max_history);
}
recent_scripts->clear(); recent_scripts->clear();
recent_scripts->add_shortcut(ED_SHORTCUT("script_editor/open_recent", TTR("Open Recent"), KEY_MASK_CMD | KEY_MASK_SHIFT | KEY_T)); recent_scripts->add_shortcut(ED_SHORTCUT("script_editor/open_recent", TTR("Open Recent"), KEY_MASK_CMD | KEY_MASK_SHIFT | KEY_T));
recent_scripts->add_separator(); recent_scripts->add_separator();
const int max_shown = 8; String path;
for (int i = 0; i < previous_scripts.size() && i <= max_shown; i++) { for (int i = 0; i < rc.size(); i++) {
String path = previous_scripts.get(i);
// just show script name and last dir path = rc[i];
recent_scripts->add_item(path.get_slice("/", path.get_slice_count("/") - 2) + "/" + path.get_file()); recent_scripts->add_item(path.replace("res://", ""));
} }
recent_scripts->add_separator(); recent_scripts->add_separator();
@ -471,7 +467,7 @@ void ScriptEditor::_open_recent_script(int p_idx) {
// clear button // clear button
if (p_idx == recent_scripts->get_item_count() - 1) { if (p_idx == recent_scripts->get_item_count() - 1) {
previous_scripts.clear(); EditorSettings::get_singleton()->set_project_metadata("recent_files", "scripts", Array());
call_deferred("_update_recent_scripts"); call_deferred("_update_recent_scripts");
return; return;
} }
@ -481,22 +477,34 @@ void ScriptEditor::_open_recent_script(int p_idx) {
p_idx -= 2; p_idx -= 2;
} }
if (p_idx < previous_scripts.size() && p_idx >= 0) { Array rc = EditorSettings::get_singleton()->get_project_metadata("recent_files", "scripts", Array());
ERR_FAIL_INDEX(p_idx, rc.size());
String path = previous_scripts.get(p_idx); String path = rc[p_idx];
// if its not on disk its a help file or deleted // if its not on disk its a help file or deleted
if (FileAccess::exists(path)) { if (FileAccess::exists(path)) {
Ref<Script> script = ResourceLoader::load(path); Ref<Script> script = ResourceLoader::load(path);
if (script.is_valid()) { if (script.is_valid()) {
edit(script, true); edit(script, true);
} return;
// if it's a path then its most likely a delted file not help
} else if (!path.is_resource_file()) {
_help_class_open(path);
} }
previous_scripts.remove(p_idx);
_update_recent_scripts(); // if it's a path then its most likely a deleted file not help
} else if (!path.is_resource_file()) {
_help_class_open(path);
return;
} }
rc.remove(p_idx);
EditorSettings::get_singleton()->set_project_metadata("recent_files", "scripts", rc);
_update_recent_scripts();
_show_error_dialog(path);
}
void ScriptEditor::_show_error_dialog(String p_path) {
error_dialog->set_text(vformat(TTR("Can't open '%s'. The file could have been moved or deleted."), p_path));
error_dialog->popup_centered_minsize();
} }
void ScriptEditor::_close_tab(int p_idx, bool p_save) { void ScriptEditor::_close_tab(int p_idx, bool p_save) {
@ -508,14 +516,10 @@ void ScriptEditor::_close_tab(int p_idx, bool p_save) {
Node *tselected = tab_container->get_child(selected); Node *tselected = tab_container->get_child(selected);
ScriptEditorBase *current = Object::cast_to<ScriptEditorBase>(tab_container->get_child(selected)); ScriptEditorBase *current = Object::cast_to<ScriptEditorBase>(tab_container->get_child(selected));
if (current) { if (current) {
_add_recent_script(current->get_edited_script()->get_path());
if (p_save) { if (p_save) {
apply_scripts(); apply_scripts();
} }
notify_script_close(current->get_edited_script()); notify_script_close(current->get_edited_script());
} else {
EditorHelp *help = Object::cast_to<EditorHelp>(tab_container->get_child(selected));
_add_recent_script(help->get_class());
} }
// roll back to previous tab // roll back to previous tab
@ -1787,6 +1791,7 @@ bool ScriptEditor::edit(const Ref<Script> &p_script, int p_line, int p_col, bool
se->goto_line(p_line - 1); se->goto_line(p_line - 1);
notify_script_changed(p_script); notify_script_changed(p_script);
_add_recent_script(p_script->get_path());
return true; return true;
} }
@ -2304,6 +2309,7 @@ void ScriptEditor::_help_class_open(const String &p_class) {
_go_to_tab(tab_container->get_tab_count() - 1); _go_to_tab(tab_container->get_tab_count() - 1);
eh->go_to_class(p_class, 0); eh->go_to_class(p_class, 0);
eh->connect("go_to_help", this, "_help_class_goto"); eh->connect("go_to_help", this, "_help_class_goto");
_add_recent_script(p_class);
_update_script_names(); _update_script_names();
_save_layout(); _save_layout();
} }
@ -2332,6 +2338,7 @@ void ScriptEditor::_help_class_goto(const String &p_desc) {
_go_to_tab(tab_container->get_tab_count() - 1); _go_to_tab(tab_container->get_tab_count() - 1);
eh->go_to_help(p_desc); eh->go_to_help(p_desc);
eh->connect("go_to_help", this, "_help_class_goto"); eh->connect("go_to_help", this, "_help_class_goto");
_add_recent_script(eh->get_class());
_update_script_names(); _update_script_names();
_save_layout(); _save_layout();
} }
@ -2738,6 +2745,10 @@ ScriptEditor::ScriptEditor(EditorNode *p_editor) {
add_child(file_dialog); add_child(file_dialog);
file_dialog->connect("file_selected", this, "_file_dialog_action"); file_dialog->connect("file_selected", this, "_file_dialog_action");
error_dialog = memnew(AcceptDialog);
add_child(error_dialog);
error_dialog->get_ok()->set_text(TTR("I see.."));
debugger = memnew(ScriptEditorDebugger(editor)); debugger = memnew(ScriptEditorDebugger(editor));
debugger->connect("goto_script_line", this, "_goto_script_line"); debugger->connect("goto_script_line", this, "_goto_script_line");
debugger->connect("show_debugger", this, "_show_debugger"); debugger->connect("show_debugger", this, "_show_debugger");

5
editor/plugins/script_editor_plugin.h
View File

@ -198,6 +198,7 @@ class ScriptEditor : public PanelContainer {
VSplitContainer *list_split; VSplitContainer *list_split;
TabContainer *tab_container; TabContainer *tab_container;
EditorFileDialog *file_dialog; EditorFileDialog *file_dialog;
AcceptDialog *error_dialog;
ConfirmationDialog *erase_tab_confirm; ConfirmationDialog *erase_tab_confirm;
ScriptCreateDialog *script_create_dialog; ScriptCreateDialog *script_create_dialog;
ScriptEditorDebugger *debugger; ScriptEditorDebugger *debugger;
@ -227,8 +228,6 @@ class ScriptEditor : public PanelContainer {
Vector<ScriptHistory> history; Vector<ScriptHistory> history;
int history_pos; int history_pos;
Vector<String> previous_scripts;
EditorHelpIndex *help_index; EditorHelpIndex *help_index;
void _tab_changed(int p_which); void _tab_changed(int p_which);
@ -250,6 +249,8 @@ class ScriptEditor : public PanelContainer {
void _update_recent_scripts(); void _update_recent_scripts();
void _open_recent_script(int p_idx); void _open_recent_script(int p_idx);
void _show_error_dialog(String p_path);
void _close_tab(int p_idx, bool p_save = true); void _close_tab(int p_idx, bool p_save = true);
void _close_current_tab(); void _close_current_tab();