asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Packages Wiki Activity Actions

Rework how script is edited when clicking icon

Browse source
This commit is contained in:
kobewi 2024-02-13 16:54:33 +01:00
parent dfe226b933
commit 8ec86e29c8
3 changed files with 31 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
editor/editor_node.cpp
View file

@ -2250,21 +2250,15 @@ void EditorNode::push_item(Object *p_object, const String &p_property, bool p_in
hide_unused_editors(); hide_unused_editors();
return; return;
} }
_add_to_history(p_object, p_property, p_inspector_only);
ObjectID id = p_object->get_instance_id();
if (id != editor_history.get_current()) {
if (p_inspector_only) {
editor_history.add_object(id, String(), true);
} else if (p_property.is_empty()) {
editor_history.add_object(id);
} else {
editor_history.add_object(id, p_property);
}
}
_edit_current(); _edit_current();
} }
void EditorNode::push_item_no_inspector(Object *p_object) {
_add_to_history(p_object, "", false);
_edit_current(false, true);
}
void EditorNode::save_default_environment() { void EditorNode::save_default_environment() {
Ref<Environment> fallback = get_tree()->get_root()->get_world_3d()->get_fallback_environment(); Ref<Environment> fallback = get_tree()->get_root()->get_world_3d()->get_fallback_environment();
@ -2328,7 +2322,20 @@ static bool overrides_external_editor(Object *p_object) {
return script->get_language()->overrides_external_editor(); return script->get_language()->overrides_external_editor();
} }
void EditorNode::_edit_current(bool p_skip_foreign) { void EditorNode::_add_to_history(const Object *p_object, const String &p_property, bool p_inspector_only) {
ObjectID id = p_object->get_instance_id();
if (id != editor_history.get_current()) {
if (p_inspector_only) {
editor_history.add_object(id, String(), true);
} else if (p_property.is_empty()) {
editor_history.add_object(id);
} else {
editor_history.add_object(id, p_property);
}
}
}
void EditorNode::_edit_current(bool p_skip_foreign, bool p_skip_inspector_update) {
ObjectID current_id = editor_history.get_current(); ObjectID current_id = editor_history.get_current();
Object *current_obj = current_id.is_valid() ? ObjectDB::get_instance(current_id) : nullptr; Object *current_obj = current_id.is_valid() ? ObjectDB::get_instance(current_id) : nullptr;
@ -2377,11 +2384,13 @@ void EditorNode::_edit_current(bool p_skip_foreign) {
Resource *current_res = Object::cast_to<Resource>(current_obj); Resource *current_res = Object::cast_to<Resource>(current_obj);
ERR_FAIL_NULL(current_res); ERR_FAIL_NULL(current_res);
if (!p_skip_inspector_update) {
InspectorDock::get_inspector_singleton()->edit(current_res); InspectorDock::get_inspector_singleton()->edit(current_res);
SceneTreeDock::get_singleton()->set_selected(nullptr); SceneTreeDock::get_singleton()->set_selected(nullptr);
NodeDock::get_singleton()->set_node(nullptr); NodeDock::get_singleton()->set_node(nullptr);
InspectorDock::get_singleton()->update(nullptr); InspectorDock::get_singleton()->update(nullptr);
ImportDock::get_singleton()->set_edit_path(current_res->get_path()); ImportDock::get_singleton()->set_edit_path(current_res->get_path());
}
int subr_idx = current_res->get_path().find("::"); int subr_idx = current_res->get_path().find("::");
if (subr_idx != -1) { if (subr_idx != -1) {

4
editor/editor_node.h
View file

@ -518,7 +518,8 @@ private:
void _dialog_action(String p_file); void _dialog_action(String p_file);
void _edit_current(bool p_skip_foreign = false); void _add_to_history(const Object *p_object, const String &p_property, bool p_inspector_only);
void _edit_current(bool p_skip_foreign = false, bool p_skip_inspector_update = false);
void _dialog_display_save_error(String p_file, Error p_error); void _dialog_display_save_error(String p_file, Error p_error);
void _dialog_display_load_error(String p_file, Error p_error); void _dialog_display_load_error(String p_file, Error p_error);
@ -752,6 +753,7 @@ public:
void show_about() { _menu_option_confirm(HELP_ABOUT, false); } void show_about() { _menu_option_confirm(HELP_ABOUT, false); }
void push_item(Object *p_object, const String &p_property = "", bool p_inspector_only = false); void push_item(Object *p_object, const String &p_property = "", bool p_inspector_only = false);
void push_item_no_inspector(Object *p_object);
void edit_item(Object *p_object, Object *p_editing_owner); void edit_item(Object *p_object, Object *p_editing_owner);
void push_node_item(Node *p_node); void push_node_item(Node *p_node);
void hide_unused_editors(const Object *p_editing_owner = nullptr); void hide_unused_editors(const Object *p_editing_owner = nullptr);

4
editor/scene_tree_dock.cpp
View file

@ -1578,9 +1578,7 @@ void SceneTreeDock::_load_request(const String &p_path) {
} }
void SceneTreeDock::_script_open_request(const Ref<Script> &p_script) { void SceneTreeDock::_script_open_request(const Ref<Script> &p_script) {
if (ScriptEditor::get_singleton()->edit(p_script, true)) { EditorNode::get_singleton()->push_item_no_inspector(p_script.ptr());
EditorNode::get_singleton()->editor_select(EditorNode::EDITOR_SCRIPT);
}
} }
void SceneTreeDock::_push_item(Object *p_object) { void SceneTreeDock::_push_item(Object *p_object) {