diff options
-rw-r--r-- | doc/classes/Button.xml | 2 | ||||
-rw-r--r-- | drivers/gles2/shader_compiler_gles2.cpp | 2 | ||||
-rw-r--r-- | editor/animation_track_editor.cpp | 9 | ||||
-rw-r--r-- | editor/groups_editor.cpp | 213 | ||||
-rw-r--r-- | editor/groups_editor.h | 20 | ||||
-rw-r--r-- | editor/plugins/asset_library_editor_plugin.cpp | 13 | ||||
-rw-r--r-- | editor/plugins/asset_library_editor_plugin.h | 1 | ||||
-rw-r--r-- | modules/gdscript/SCsub | 11 | ||||
-rw-r--r-- | modules/gdscript/gdscript_parser.cpp | 2 | ||||
-rw-r--r-- | modules/gdscript/register_types.cpp | 15 | ||||
-rw-r--r-- | platform/x11/os_x11.cpp | 5 | ||||
-rw-r--r-- | scene/gui/button.cpp | 6 |
12 files changed, 208 insertions, 91 deletions
diff --git a/doc/classes/Button.xml b/doc/classes/Button.xml index adf826c26b..6a8cdcd2a8 100644 --- a/doc/classes/Button.xml +++ b/doc/classes/Button.xml @@ -15,7 +15,7 @@ Text alignment policy for the button's text, use one of the [code]ALIGN_*[/code] constants. </member> <member name="clip_text" type="bool" setter="set_clip_text" getter="get_clip_text" default="false"> - When this property is enabled, text that is too large to fit the button is clipped, when disabled the Button will always be wide enough to hold the text. This property is disabled by default. + When this property is enabled, text that is too large to fit the button is clipped, when disabled the Button will always be wide enough to hold the text. </member> <member name="flat" type="bool" setter="set_flat" getter="is_flat" default="false"> Flat buttons don't display decoration. diff --git a/drivers/gles2/shader_compiler_gles2.cpp b/drivers/gles2/shader_compiler_gles2.cpp index ab0ee7c24f..640d45ae65 100644 --- a/drivers/gles2/shader_compiler_gles2.cpp +++ b/drivers/gles2/shader_compiler_gles2.cpp @@ -919,7 +919,7 @@ ShaderCompilerGLES2::ShaderCompilerGLES2() { actions[VS::SHADER_CANVAS_ITEM].renames["WORLD_MATRIX"] = "modelview_matrix"; actions[VS::SHADER_CANVAS_ITEM].renames["PROJECTION_MATRIX"] = "projection_matrix"; - actions[VS::SHADER_CANVAS_ITEM].renames["EXTRA_MATRIX"] = "extra_matrix"; + actions[VS::SHADER_CANVAS_ITEM].renames["EXTRA_MATRIX"] = "extra_matrix_instance"; actions[VS::SHADER_CANVAS_ITEM].renames["TIME"] = "time"; actions[VS::SHADER_CANVAS_ITEM].renames["AT_LIGHT_PASS"] = "at_light_pass"; actions[VS::SHADER_CANVAS_ITEM].renames["INSTANCE_CUSTOM"] = "instance_custom"; diff --git a/editor/animation_track_editor.cpp b/editor/animation_track_editor.cpp index c93dc2b340..a163490cfb 100644 --- a/editor/animation_track_editor.cpp +++ b/editor/animation_track_editor.cpp @@ -1799,6 +1799,13 @@ void AnimationTimelineEdit::_gui_input(const Ref<InputEvent> &p_event) { if (mm.is_valid()) { + if (hsize_rect.has_point(mm->get_position())) { + // Change the cursor to indicate that the track name column's width can be adjusted + set_default_cursor_shape(Control::CURSOR_HSIZE); + } else { + set_default_cursor_shape(Control::CURSOR_ARROW); + } + if (dragging_hsize) { int ofs = mm->get_position().x - dragging_hsize_from; name_limit = dragging_hsize_at + ofs; @@ -1857,7 +1864,7 @@ AnimationTimelineEdit::AnimationTimelineEdit() { use_fps = false; editing = false; - name_limit = 150; + name_limit = 150 * EDSCALE; zoom = NULL; play_position_pos = 0; diff --git a/editor/groups_editor.cpp b/editor/groups_editor.cpp index 5a8dc205c2..4cefb53617 100644 --- a/editor/groups_editor.cpp +++ b/editor/groups_editor.cpp @@ -35,12 +35,6 @@ #include "scene/gui/label.h" #include "scene/resources/packed_scene.h" -void GroupDialog::ok_pressed() { -} - -void GroupDialog::_cancel_pressed() { -} - void GroupDialog::_group_selected() { nodes_to_add->clear(); add_node_root = nodes_to_add->create_item(); @@ -49,11 +43,14 @@ void GroupDialog::_group_selected() { remove_node_root = nodes_to_remove->create_item(); if (!groups->is_anything_selected()) { + group_empty->hide(); return; } selected_group = groups->get_selected()->get_text(0); _load_nodes(scene_tree->get_edited_scene_root()); + + group_empty->set_visible(!remove_node_root->get_children()); } void GroupDialog::_load_nodes(Node *p_current) { @@ -129,15 +126,26 @@ void GroupDialog::_add_pressed() { return; } + undo_redo->create_action(TTR("Add to Group")); + while (selected) { Node *node = scene_tree->get_edited_scene_root()->get_node(selected->get_metadata(0)); - node->add_to_group(selected_group, true); + undo_redo->add_do_method(node, "add_to_group", selected_group, true); + undo_redo->add_undo_method(node, "remove_from_group", selected_group); selected = nodes_to_add->get_next_selected(selected); } - _group_selected(); - EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor()->update_tree(); + undo_redo->add_do_method(this, "_group_selected"); + undo_redo->add_undo_method(this, "_group_selected"); + undo_redo->add_do_method(this, "emit_signal", "group_edited"); + undo_redo->add_undo_method(this, "emit_signal", "group_edited"); + + // To force redraw of scene tree. + undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + + undo_redo->commit_action(); } void GroupDialog::_removed_pressed() { @@ -147,15 +155,26 @@ void GroupDialog::_removed_pressed() { return; } + undo_redo->create_action(TTR("Remove from Group")); + while (selected) { Node *node = scene_tree->get_edited_scene_root()->get_node(selected->get_metadata(0)); - node->remove_from_group(selected_group); + undo_redo->add_do_method(node, "remove_from_group", selected_group); + undo_redo->add_undo_method(node, "add_to_group", selected_group, true); selected = nodes_to_add->get_next_selected(selected); } - _group_selected(); - EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor()->update_tree(); + undo_redo->add_do_method(this, "_group_selected"); + undo_redo->add_undo_method(this, "_group_selected"); + undo_redo->add_do_method(this, "emit_signal", "group_edited"); + undo_redo->add_undo_method(this, "emit_signal", "group_edited"); + + // To force redraw of scene tree. + undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + + undo_redo->commit_action(); } void GroupDialog::_remove_filter_changed(const String &p_filter) { @@ -166,11 +185,29 @@ void GroupDialog::_add_filter_changed(const String &p_filter) { _group_selected(); } -void GroupDialog::_add_group_pressed() { +void GroupDialog::_add_group_pressed(const String &p_name) { _add_group(add_group_text->get_text()); add_group_text->clear(); } +void GroupDialog::_add_group(String p_name) { + if (!is_visible()) { + return; // No need to edit the dialog if it's not being used. + } + + String name = p_name.strip_edges(); + if (name == "" || groups->search_item_text(name)) { + return; + } + + TreeItem *new_group = groups->create_item(groups_root); + new_group->set_text(0, name); + new_group->add_button(0, get_icon("Remove", "EditorIcons"), 0); + new_group->set_editable(0, true); + new_group->select(0); + groups->ensure_cursor_is_visible(); +} + void GroupDialog::_group_renamed() { TreeItem *renamed_group = groups->get_edited(); if (!renamed_group) { @@ -194,38 +231,51 @@ void GroupDialog::_group_renamed() { return; } + undo_redo->create_action(TTR("Rename Group")); + List<Node *> nodes; scene_tree->get_nodes_in_group(selected_group, &nodes); bool removed_all = true; for (List<Node *>::Element *E = nodes.front(); E; E = E->next()) { Node *node = E->get(); if (_can_edit(node, selected_group)) { - node->remove_from_group(selected_group); - node->add_to_group(name, true); + undo_redo->add_do_method(node, "remove_from_group", selected_group); + undo_redo->add_undo_method(node, "remove_from_group", name); + undo_redo->add_do_method(node, "add_to_group", name, true); + undo_redo->add_undo_method(node, "add_to_group", selected_group, true); } else { removed_all = false; } } if (!removed_all) { - _add_group(selected_group); + undo_redo->add_do_method(this, "_add_group", selected_group); + undo_redo->add_undo_method(this, "_delete_group_item", selected_group); } - selected_group = renamed_group->get_text(0); - _group_selected(); -} + undo_redo->add_do_method(this, "_rename_group_item", selected_group, renamed_group->get_text(0)); + undo_redo->add_undo_method(this, "_rename_group_item", renamed_group->get_text(0), selected_group); + undo_redo->add_do_method(this, "_group_selected"); + undo_redo->add_undo_method(this, "_group_selected"); + undo_redo->add_do_method(this, "emit_signal", "group_edited"); + undo_redo->add_undo_method(this, "emit_signal", "group_edited"); -void GroupDialog::_add_group(String p_name) { + undo_redo->commit_action(); +} - String name = p_name.strip_edges(); - if (name == "" || groups->search_item_text(name)) { - return; +void GroupDialog::_rename_group_item(const String &p_old_name, const String &p_new_name) { + if (!is_visible()) { + return; // No need to edit the dialog if it's not being used. } - TreeItem *new_group = groups->create_item(groups_root); - new_group->set_text(0, name); - new_group->add_button(0, get_icon("Remove", "EditorIcons"), 0); - new_group->set_editable(0, true); + selected_group = p_new_name; + + for (TreeItem *E = groups_root->get_children(); E; E = E->get_next()) { + if (E->get_text(0) == p_old_name) { + E->set_text(0, p_new_name); + return; + } + } } void GroupDialog::_load_groups(Node *p_current) { @@ -251,29 +301,57 @@ void GroupDialog::_delete_group_pressed(Object *p_item, int p_column, int p_id) String name = ti->get_text(0); + undo_redo->create_action(TTR("Delete Group")); + List<Node *> nodes; scene_tree->get_nodes_in_group(name, &nodes); bool removed_all = true; for (List<Node *>::Element *E = nodes.front(); E; E = E->next()) { if (_can_edit(E->get(), name)) { - E->get()->remove_from_group(name); + undo_redo->add_do_method(E->get(), "remove_from_group", name); + undo_redo->add_undo_method(E->get(), "add_to_group", name, true); } else { removed_all = false; } } if (removed_all) { - if (selected_group == name) { - add_filter->clear(); - remove_filter->clear(); - nodes_to_remove->clear(); - nodes_to_add->clear(); - groups->deselect_all(); - selected_group = ""; + undo_redo->add_do_method(this, "_delete_group_item", name); + undo_redo->add_undo_method(this, "_add_group", name); + } + + undo_redo->add_do_method(this, "_group_selected"); + undo_redo->add_undo_method(this, "_group_selected"); + undo_redo->add_do_method(this, "emit_signal", "group_edited"); + undo_redo->add_undo_method(this, "emit_signal", "group_edited"); + + // To force redraw of scene tree. + undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + + undo_redo->commit_action(); +} + +void GroupDialog::_delete_group_item(const String &p_name) { + if (!is_visible()) { + return; // No need to edit the dialog if it's not being used. + } + + if (selected_group == p_name) { + add_filter->clear(); + remove_filter->clear(); + nodes_to_remove->clear(); + nodes_to_add->clear(); + groups->deselect_all(); + selected_group = ""; + } + + for (TreeItem *E = groups_root->get_children(); E; E = E->get_next()) { + if (E->get_text(0) == p_name) { + groups_root->remove_child(E); + return; } - groups_root->remove_child(ti); } - EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor()->update_tree(); } void GroupDialog::_notification(int p_what) { @@ -291,8 +369,7 @@ void GroupDialog::_notification(int p_what) { } void GroupDialog::edit() { - - popup_centered(Size2(600, 400)); + popup_centered(); groups->clear(); groups_root = groups->create_item(); @@ -308,27 +385,32 @@ void GroupDialog::edit() { } void GroupDialog::_bind_methods() { - ClassDB::bind_method("_cancel", &GroupDialog::_cancel_pressed); - ClassDB::bind_method("_add_pressed", &GroupDialog::_add_pressed); ClassDB::bind_method("_removed_pressed", &GroupDialog::_removed_pressed); ClassDB::bind_method("_delete_group_pressed", &GroupDialog::_delete_group_pressed); + ClassDB::bind_method("_delete_group_item", &GroupDialog::_delete_group_item); ClassDB::bind_method("_group_selected", &GroupDialog::_group_selected); ClassDB::bind_method("_add_group_pressed", &GroupDialog::_add_group_pressed); + ClassDB::bind_method("_add_group", &GroupDialog::_add_group); ClassDB::bind_method("_add_filter_changed", &GroupDialog::_add_filter_changed); ClassDB::bind_method("_remove_filter_changed", &GroupDialog::_remove_filter_changed); ClassDB::bind_method("_group_renamed", &GroupDialog::_group_renamed); + ClassDB::bind_method("_rename_group_item", &GroupDialog::_rename_group_item); + + ADD_SIGNAL(MethodInfo("group_edited")); } GroupDialog::GroupDialog() { + set_custom_minimum_size(Size2(600, 400)); scene_tree = SceneTree::get_singleton(); VBoxContainer *vbc = memnew(VBoxContainer); add_child(vbc); + vbc->set_anchors_and_margins_preset(PRESET_WIDE, PRESET_MODE_KEEP_SIZE, 8 * EDSCALE); HBoxContainer *hbc = memnew(HBoxContainer); vbc->add_child(hbc); @@ -345,10 +427,11 @@ GroupDialog::GroupDialog() { groups = memnew(Tree); vbc_left->add_child(groups); groups->set_hide_root(true); - groups->set_v_size_flags(SIZE_EXPAND_FILL); groups->set_select_mode(Tree::SELECT_SINGLE); groups->set_allow_reselect(true); groups->set_allow_rmb_select(true); + groups->set_v_size_flags(SIZE_EXPAND_FILL); + groups->add_constant_override("draw_guides", 1); groups->connect("item_selected", this, "_group_selected"); groups->connect("button_pressed", this, "_delete_group_pressed"); groups->connect("item_edited", this, "_group_renamed"); @@ -360,26 +443,28 @@ GroupDialog::GroupDialog() { add_group_text = memnew(LineEdit); chbc->add_child(add_group_text); add_group_text->set_h_size_flags(SIZE_EXPAND_FILL); + add_group_text->connect("text_entered", this, "_add_group_pressed"); Button *add_group_button = memnew(Button); add_group_button->set_text("Add"); chbc->add_child(add_group_button); - add_group_button->connect("pressed", this, "_add_group_pressed"); + add_group_button->connect("pressed", this, "_add_group_pressed", varray(String())); VBoxContainer *vbc_add = memnew(VBoxContainer); hbc->add_child(vbc_add); vbc_add->set_h_size_flags(SIZE_EXPAND_FILL); Label *out_of_group_title = memnew(Label); - out_of_group_title->set_text(TTR("Nodes not in Group")); + out_of_group_title->set_text(TTR("Nodes Not in Group")); vbc_add->add_child(out_of_group_title); nodes_to_add = memnew(Tree); vbc_add->add_child(nodes_to_add); nodes_to_add->set_hide_root(true); nodes_to_add->set_hide_folding(true); - nodes_to_add->set_v_size_flags(SIZE_EXPAND_FILL); nodes_to_add->set_select_mode(Tree::SELECT_MULTI); + nodes_to_add->set_v_size_flags(SIZE_EXPAND_FILL); + nodes_to_add->add_constant_override("draw_guides", 1); nodes_to_add->connect("item_selected", this, "_nodes_to_add_selected"); HBoxContainer *add_filter_hbc = memnew(HBoxContainer); @@ -426,6 +511,7 @@ GroupDialog::GroupDialog() { nodes_to_remove->set_hide_root(true); nodes_to_remove->set_hide_folding(true); nodes_to_remove->set_select_mode(Tree::SELECT_MULTI); + nodes_to_remove->add_constant_override("draw_guides", 1); nodes_to_remove->connect("item_selected", this, "_node_to_remove_selected"); HBoxContainer *remove_filter_hbc = memnew(HBoxContainer); @@ -438,8 +524,15 @@ GroupDialog::GroupDialog() { remove_filter_hbc->add_child(remove_filter); remove_filter->connect("text_changed", this, "_remove_filter_changed"); - set_title("Group Editor"); - get_cancel()->hide(); + group_empty = memnew(Label()); + group_empty->set_text(TTR("Empty groups will be automatically removed.")); + group_empty->set_valign(Label::VALIGN_CENTER); + group_empty->set_align(Label::ALIGN_CENTER); + group_empty->set_autowrap(true); + nodes_to_remove->add_child(group_empty); + group_empty->set_anchors_and_margins_preset(PRESET_WIDE, PRESET_MODE_KEEP_SIZE, 8 * EDSCALE); + + set_title(TTR("Group Editor")); set_as_toplevel(true); set_resizable(true); @@ -465,11 +558,13 @@ void GroupsEditor::_add_group(const String &p_group) { undo_redo->create_action(TTR("Add to Group")); undo_redo->add_do_method(node, "add_to_group", name, true); - undo_redo->add_do_method(this, "update_tree"); undo_redo->add_undo_method(node, "remove_from_group", name); + undo_redo->add_do_method(this, "update_tree"); undo_redo->add_undo_method(this, "update_tree"); - undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); //to force redraw of scene tree - undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); //to force redraw of scene tree + + // To force redraw of scene tree. + undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); undo_redo->commit_action(); @@ -490,11 +585,13 @@ void GroupsEditor::_remove_group(Object *p_item, int p_column, int p_id) { undo_redo->create_action(TTR("Remove from Group")); undo_redo->add_do_method(node, "remove_from_group", name); - undo_redo->add_do_method(this, "update_tree"); undo_redo->add_undo_method(node, "add_to_group", name, true); + undo_redo->add_do_method(this, "update_tree"); undo_redo->add_undo_method(this, "update_tree"); - undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); //to force redraw of scene tree - undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); //to force redraw of scene tree + + // To force redraw of scene tree. + undo_redo->add_do_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); + undo_redo->add_undo_method(EditorNode::get_singleton()->get_scene_tree_dock()->get_tree_editor(), "update_tree"); undo_redo->commit_action(); } @@ -562,11 +659,9 @@ void GroupsEditor::set_current(Node *p_node) { } void GroupsEditor::_show_group_dialog() { - group_dialog->edit(); -} -void GroupsEditor::_group_dialog_closed() { - update_tree(); + group_dialog->edit(); + group_dialog->set_undo_redo(undo_redo); } void GroupsEditor::_bind_methods() { @@ -576,7 +671,6 @@ void GroupsEditor::_bind_methods() { ClassDB::bind_method("update_tree", &GroupsEditor::update_tree); ClassDB::bind_method("_show_group_dialog", &GroupsEditor::_show_group_dialog); - ClassDB::bind_method("_group_dialog_closed", &GroupsEditor::_group_dialog_closed); } GroupsEditor::GroupsEditor() { @@ -588,7 +682,7 @@ GroupsEditor::GroupsEditor() { group_dialog = memnew(GroupDialog); group_dialog->set_as_toplevel(true); add_child(group_dialog); - group_dialog->connect("popup_hide", this, "_group_dialog_closed"); + group_dialog->connect("group_edited", this, "update_tree"); Button *group_dialog_button = memnew(Button); group_dialog_button->set_text(TTR("Manage Groups")); @@ -613,6 +707,7 @@ GroupsEditor::GroupsEditor() { tree->set_v_size_flags(SIZE_EXPAND_FILL); vbc->add_child(tree); tree->connect("button_pressed", this, "_remove_group"); + tree->add_constant_override("draw_guides", 1); add_constant_override("separation", 3 * EDSCALE); } diff --git a/editor/groups_editor.h b/editor/groups_editor.h index 4ffeea84e7..78ef99d5c8 100644 --- a/editor/groups_editor.h +++ b/editor/groups_editor.h @@ -41,13 +41,9 @@ #include "scene/gui/tool_button.h" #include "scene/gui/tree.h" -/** -@author Juan Linietsky <reduzio@gmail.com> -*/ +class GroupDialog : public WindowDialog { -class GroupDialog : public ConfirmationDialog { - - GDCLASS(GroupDialog, ConfirmationDialog); + GDCLASS(GroupDialog, WindowDialog); ConfirmationDialog *error; @@ -66,13 +62,15 @@ class GroupDialog : public ConfirmationDialog { TreeItem *remove_node_root; LineEdit *remove_filter; + Label *group_empty; + ToolButton *add_button; ToolButton *remove_button; String selected_group; - void ok_pressed(); - void _cancel_pressed(); + UndoRedo *undo_redo; + void _group_selected(); void _remove_filter_changed(const String &p_filter); @@ -80,12 +78,14 @@ class GroupDialog : public ConfirmationDialog { void _add_pressed(); void _removed_pressed(); - void _add_group_pressed(); + void _add_group_pressed(const String &p_name); void _group_renamed(); + void _rename_group_item(const String &p_old_name, const String &p_new_name); void _add_group(String p_name); void _delete_group_pressed(Object *p_item, int p_column, int p_id); + void _delete_group_item(const String &p_name); bool _can_edit(Node *p_node, String p_group); @@ -98,6 +98,7 @@ protected: public: void edit(); + void set_undo_redo(UndoRedo *p_undoredo) { undo_redo = p_undoredo; } GroupDialog(); }; @@ -122,7 +123,6 @@ class GroupsEditor : public VBoxContainer { void _close(); void _show_group_dialog(); - void _group_dialog_closed(); protected: static void _bind_methods(); diff --git a/editor/plugins/asset_library_editor_plugin.cpp b/editor/plugins/asset_library_editor_plugin.cpp index fecab89d23..cb68f5eaaf 100644 --- a/editor/plugins/asset_library_editor_plugin.cpp +++ b/editor/plugins/asset_library_editor_plugin.cpp @@ -196,14 +196,15 @@ void EditorAssetLibraryItemDescription::set_image(int p_type, int p_index, const } break; } } + void EditorAssetLibraryItemDescription::_notification(int p_what) { switch (p_what) { case NOTIFICATION_ENTER_TREE: { previews_bg->add_style_override("panel", get_stylebox("normal", "TextEdit")); - desc_bg->add_style_override("panel", get_stylebox("normal", "TextEdit")); } break; } } + void EditorAssetLibraryItemDescription::_bind_methods() { ClassDB::bind_method(D_METHOD("set_image"), &EditorAssetLibraryItemDescription::set_image); ClassDB::bind_method(D_METHOD("_link_click"), &EditorAssetLibraryItemDescription::_link_click); @@ -283,16 +284,12 @@ EditorAssetLibraryItemDescription::EditorAssetLibraryItemDescription() { item = memnew(EditorAssetLibraryItem); desc_vbox->add_child(item); - desc_vbox->set_custom_minimum_size(Size2(300 * EDSCALE, 0)); - - desc_bg = memnew(PanelContainer); - desc_vbox->add_child(desc_bg); - desc_bg->set_v_size_flags(SIZE_EXPAND_FILL); + desc_vbox->set_custom_minimum_size(Size2(440 * EDSCALE, 0)); description = memnew(RichTextLabel); - desc_bg->add_child(description); + desc_vbox->add_child(description); + description->set_v_size_flags(SIZE_EXPAND_FILL); description->connect("meta_clicked", this, "_link_click"); - description->set_custom_minimum_size(Size2(440 * EDSCALE, 300 * EDSCALE)); VBoxContainer *previews_vbox = memnew(VBoxContainer); hbox->add_child(previews_vbox); diff --git a/editor/plugins/asset_library_editor_plugin.h b/editor/plugins/asset_library_editor_plugin.h index d81606c284..b17a6dfe54 100644 --- a/editor/plugins/asset_library_editor_plugin.h +++ b/editor/plugins/asset_library_editor_plugin.h @@ -91,7 +91,6 @@ class EditorAssetLibraryItemDescription : public ConfirmationDialog { ScrollContainer *previews; HBoxContainer *preview_hb; PanelContainer *previews_bg; - PanelContainer *desc_bg; struct Preview { int id; diff --git a/modules/gdscript/SCsub b/modules/gdscript/SCsub index 6285e6bb54..74e653ce43 100644 --- a/modules/gdscript/SCsub +++ b/modules/gdscript/SCsub @@ -8,5 +8,12 @@ env_gdscript = env_modules.Clone() env_gdscript.add_source_files(env.modules_sources, "*.cpp") if env['tools']: - env_gdscript.add_source_files(env.modules_sources, "./editor/*.cpp") - env_gdscript.add_source_files(env.modules_sources, "./language_server/*.cpp") + env_gdscript.add_source_files(env.modules_sources, "./editor/*.cpp") + + # Those two modules are required for the language server protocol + if env['module_jsonrpc_enabled'] and env['module_websocket_enabled']: + env_gdscript.add_source_files(env.modules_sources, "./language_server/*.cpp") + else: + # Using a define in the disabled case, to avoid having an extra define + # in regular builds where all modules are enabled. + env_gdscript.Append(CPPDEFINES=['GDSCRIPT_NO_LSP']) diff --git a/modules/gdscript/gdscript_parser.cpp b/modules/gdscript/gdscript_parser.cpp index a9f22225a0..99bfdae7ec 100644 --- a/modules/gdscript/gdscript_parser.cpp +++ b/modules/gdscript/gdscript_parser.cpp @@ -3366,7 +3366,7 @@ void GDScriptParser::_parse_extends(ClassNode *p_class) { return; } - if (!p_class->constant_expressions.empty() || !p_class->subclasses.empty() || !p_class->functions.empty() || !p_class->variables.empty() || p_class->classname_used) { + if (!p_class->constant_expressions.empty() || !p_class->subclasses.empty() || !p_class->functions.empty() || !p_class->variables.empty()) { _set_error("\"extends\" must be used before anything else."); return; diff --git a/modules/gdscript/register_types.cpp b/modules/gdscript/register_types.cpp index d07949b34b..94b9e8c2d9 100644 --- a/modules/gdscript/register_types.cpp +++ b/modules/gdscript/register_types.cpp @@ -34,10 +34,8 @@ #include "core/io/resource_loader.h" #include "core/os/dir_access.h" #include "core/os/file_access.h" -#include "editor/gdscript_highlighter.h" #include "gdscript.h" #include "gdscript_tokenizer.h" -#include "language_server/gdscript_language_server.h" GDScriptLanguage *script_language_gd = NULL; Ref<ResourceFormatLoaderGDScript> resource_loader_gd; @@ -45,10 +43,15 @@ Ref<ResourceFormatSaverGDScript> resource_saver_gd; #ifdef TOOLS_ENABLED -#include "core/engine.h" #include "editor/editor_export.h" #include "editor/editor_node.h" #include "editor/editor_settings.h" +#include "editor/gdscript_highlighter.h" + +#ifndef GDSCRIPT_NO_LSP +#include "core/engine.h" +#include "language_server/gdscript_language_server.h" +#endif // !GDSCRIPT_NO_LSP class EditorExportGDScript : public EditorExportPlugin { @@ -137,13 +140,15 @@ static void _editor_init() { gd_export.instance(); EditorExport::get_singleton()->add_export_plugin(gd_export); +#ifndef GDSCRIPT_NO_LSP register_lsp_types(); GDScriptLanguageServer *lsp_plugin = memnew(GDScriptLanguageServer); EditorNode::get_singleton()->add_editor_plugin(lsp_plugin); Engine::get_singleton()->add_singleton(Engine::Singleton("GDScriptLanguageProtocol", GDScriptLanguageProtocol::get_singleton())); +#endif // !GDSCRIPT_NO_LSP } -#endif +#endif // TOOLS_ENABLED void register_gdscript_types() { @@ -162,7 +167,7 @@ void register_gdscript_types() { #ifdef TOOLS_ENABLED ScriptEditor::register_create_syntax_highlighter_function(GDScriptSyntaxHighlighter::create); EditorNode::add_init_callback(_editor_init); -#endif +#endif // TOOLS_ENABLED } void unregister_gdscript_types() { diff --git a/platform/x11/os_x11.cpp b/platform/x11/os_x11.cpp index ca72393e43..dfa0a45538 100644 --- a/platform/x11/os_x11.cpp +++ b/platform/x11/os_x11.cpp @@ -1757,7 +1757,10 @@ void OS_X11::handle_key_event(XKeyEvent *p_event, bool p_echo) { // XLookupString returns keysyms usable as nice scancodes/ char str[256 + 1]; - XLookupString(xkeyevent, str, 256, &keysym_keycode, NULL); + XKeyEvent xkeyevent_no_mod = *xkeyevent; + xkeyevent_no_mod.state &= ~ShiftMask; + xkeyevent_no_mod.state &= ~ControlMask; + XLookupString(&xkeyevent_no_mod, str, 256, &keysym_keycode, NULL); // Meanwhile, XLookupString returns keysyms useful for unicode. diff --git a/scene/gui/button.cpp b/scene/gui/button.cpp index 65e9cccd05..4ce3f18505 100644 --- a/scene/gui/button.cpp +++ b/scene/gui/button.cpp @@ -159,7 +159,11 @@ void Button::_notification(int p_what) { switch (align) { case ALIGN_LEFT: { - text_ofs.x = style->get_margin(MARGIN_LEFT) + icon_ofs.x + _internal_margin[MARGIN_LEFT] + get_constant("hseparation"); + if (_internal_margin[MARGIN_LEFT] > 0) { + text_ofs.x = style->get_margin(MARGIN_LEFT) + icon_ofs.x + _internal_margin[MARGIN_LEFT] + get_constant("hseparation"); + } else { + text_ofs.x = style->get_margin(MARGIN_LEFT) + icon_ofs.x; + } text_ofs.y += style->get_offset().y; } break; case ALIGN_CENTER: { |