Fix toggle scripts panel to allow using shortcut in other areas

This commit is contained in:
Yuri Roubinsky 2019-12-17 16:51:49 +03:00
parent accf68b2cc
commit 03928c5b36
5 changed files with 32 additions and 8 deletions

View file

@ -1509,7 +1509,7 @@ void CodeTextEditor::_set_show_warnings_panel(bool p_show) {
} }
void CodeTextEditor::_toggle_scripts_pressed() { void CodeTextEditor::_toggle_scripts_pressed() {
toggle_scripts_button->set_icon(ScriptEditor::get_singleton()->toggle_scripts_panel() ? get_icon("Back", "EditorIcons") : get_icon("Forward", "EditorIcons")); toggle_scripts_button->set_icon(ScriptEditor::get_singleton()->toggle_scripts_panel(this) ? get_icon("Back", "EditorIcons") : get_icon("Forward", "EditorIcons"));
} }
void CodeTextEditor::_error_pressed(const Ref<InputEvent> &p_event) { void CodeTextEditor::_error_pressed(const Ref<InputEvent> &p_event) {
@ -1528,7 +1528,7 @@ void CodeTextEditor::_notification(int p_what) {
} break; } break;
case NOTIFICATION_THEME_CHANGED: { case NOTIFICATION_THEME_CHANGED: {
if (toggle_scripts_button->is_visible()) { if (toggle_scripts_button->is_visible()) {
toggle_scripts_button->set_icon(ScriptEditor::get_singleton()->is_scripts_panel_toggled() ? get_icon("Back", "EditorIcons") : get_icon("Forward", "EditorIcons")); update_toggle_scripts_button();
} }
_update_font(); _update_font();
} break; } break;
@ -1537,6 +1537,9 @@ void CodeTextEditor::_notification(int p_what) {
add_constant_override("separation", 4 * EDSCALE); add_constant_override("separation", 4 * EDSCALE);
} break; } break;
case NOTIFICATION_VISIBILITY_CHANGED: { case NOTIFICATION_VISIBILITY_CHANGED: {
if (toggle_scripts_button->is_visible()) {
update_toggle_scripts_button();
}
set_process_input(is_visible_in_tree()); set_process_input(is_visible_in_tree());
} break; } break;
default: default:
@ -1649,6 +1652,10 @@ void CodeTextEditor::show_toggle_scripts_button() {
toggle_scripts_button->show(); toggle_scripts_button->show();
} }
void CodeTextEditor::update_toggle_scripts_button() {
toggle_scripts_button->set_icon(ScriptEditor::get_singleton()->is_scripts_panel_toggled() ? get_icon("Back", "EditorIcons") : get_icon("Forward", "EditorIcons"));
}
CodeTextEditor::CodeTextEditor() { CodeTextEditor::CodeTextEditor() {
code_complete_func = NULL; code_complete_func = NULL;
@ -1693,7 +1700,7 @@ CodeTextEditor::CodeTextEditor() {
toggle_scripts_button = memnew(ToolButton); toggle_scripts_button = memnew(ToolButton);
toggle_scripts_button->connect("pressed", this, "_toggle_scripts_pressed"); toggle_scripts_button->connect("pressed", this, "_toggle_scripts_pressed");
status_bar->add_child(toggle_scripts_button); status_bar->add_child(toggle_scripts_button);
toggle_scripts_button->set_shortcut(ED_SHORTCUT("script_editor/toggle_scripts_panel", TTR("Toggle Scripts Panel"), KEY_MASK_CMD | KEY_BACKSLASH)); toggle_scripts_button->set_tooltip(TTR("Toggle Scripts Panel") + " (" + ED_GET_SHORTCUT("script_editor/toggle_scripts_panel")->get_as_text() + ")");
toggle_scripts_button->hide(); toggle_scripts_button->hide();
// Error // Error

View file

@ -256,6 +256,7 @@ public:
void validate_script(); void validate_script();
void show_toggle_scripts_button(); void show_toggle_scripts_button();
void update_toggle_scripts_button();
CodeTextEditor(); CodeTextEditor();
}; };

View file

@ -987,8 +987,11 @@ Array ScriptEditor::_get_open_scripts() const {
return ret; return ret;
} }
bool ScriptEditor::toggle_scripts_panel() { bool ScriptEditor::toggle_scripts_panel(CodeTextEditor *p_editor) {
list_split->set_visible(!list_split->is_visible()); list_split->set_visible(!list_split->is_visible());
if (p_editor) {
p_editor->update_toggle_scripts_button();
}
return list_split->is_visible(); return list_split->is_visible();
} }
@ -998,6 +1001,7 @@ bool ScriptEditor::is_scripts_panel_toggled() {
void ScriptEditor::_menu_option(int p_option) { void ScriptEditor::_menu_option(int p_option) {
ScriptEditorBase *current = _get_current_editor();
switch (p_option) { switch (p_option) {
case FILE_NEW: { case FILE_NEW: {
script_create_dialog->config("Node", "new_script"); script_create_dialog->config("Node", "new_script");
@ -1136,11 +1140,19 @@ void ScriptEditor::_menu_option(int p_option) {
debug_menu->get_popup()->set_item_checked(debug_menu->get_popup()->get_item_index(DEBUG_WITH_EXTERNAL_EDITOR), debug_with_external_editor); debug_menu->get_popup()->set_item_checked(debug_menu->get_popup()->get_item_index(DEBUG_WITH_EXTERNAL_EDITOR), debug_with_external_editor);
} break; } break;
case TOGGLE_SCRIPTS_PANEL: { case TOGGLE_SCRIPTS_PANEL: {
toggle_scripts_panel(); if (current) {
CodeTextEditor *code_editor = NULL;
ScriptTextEditor *editor = dynamic_cast<ScriptTextEditor *>(current);
if (editor) {
code_editor = editor->code_editor;
}
toggle_scripts_panel(code_editor);
} else {
toggle_scripts_panel(NULL);
}
} }
} }
ScriptEditorBase *current = _get_current_editor();
if (current) { if (current) {
switch (p_option) { switch (p_option) {
@ -3315,6 +3327,9 @@ ScriptEditor::ScriptEditor(EditorNode *p_editor) {
file_menu->get_popup()->add_separator(); file_menu->get_popup()->add_separator();
file_menu->get_popup()->add_shortcut(ED_SHORTCUT("script_editor/run_file", TTR("Run"), KEY_MASK_CMD | KEY_MASK_SHIFT | KEY_X), FILE_RUN); file_menu->get_popup()->add_shortcut(ED_SHORTCUT("script_editor/run_file", TTR("Run"), KEY_MASK_CMD | KEY_MASK_SHIFT | KEY_X), FILE_RUN);
file_menu->get_popup()->add_separator();
file_menu->get_popup()->add_shortcut(ED_SHORTCUT("script_editor/toggle_scripts_panel", TTR("Toggle Scripts Panel"), KEY_MASK_CMD | KEY_BACKSLASH), TOGGLE_SCRIPTS_PANEL);
file_menu->get_popup()->connect("id_pressed", this, "_menu_option"); file_menu->get_popup()->connect("id_pressed", this, "_menu_option");
script_search_menu = memnew(MenuButton); script_search_menu = memnew(MenuButton);

View file

@ -419,7 +419,7 @@ protected:
public: public:
static ScriptEditor *get_singleton() { return script_editor; } static ScriptEditor *get_singleton() { return script_editor; }
bool toggle_scripts_panel(); bool toggle_scripts_panel(CodeTextEditor *p_editor);
bool is_scripts_panel_toggled(); bool is_scripts_panel_toggled();
void ensure_focus_current(); void ensure_focus_current();
void apply_scripts() const; void apply_scripts() const;

View file

@ -55,7 +55,6 @@ class ScriptTextEditor : public ScriptEditorBase {
GDCLASS(ScriptTextEditor, ScriptEditorBase); GDCLASS(ScriptTextEditor, ScriptEditorBase);
CodeTextEditor *code_editor;
RichTextLabel *warnings_panel; RichTextLabel *warnings_panel;
Ref<Script> script; Ref<Script> script;
@ -187,6 +186,8 @@ protected:
void drop_data_fw(const Point2 &p_point, const Variant &p_data, Control *p_from); void drop_data_fw(const Point2 &p_point, const Variant &p_data, Control *p_from);
public: public:
CodeTextEditor *code_editor;
void _update_connected_methods(); void _update_connected_methods();
virtual void add_syntax_highlighter(SyntaxHighlighter *p_highlighter); virtual void add_syntax_highlighter(SyntaxHighlighter *p_highlighter);