summaryrefslogtreecommitdiffstats
path: root/modules/visual_script/visual_script_editor.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Add descriptive tooltips in VS editorEmmanuel Barroga2019-07-311-4/+4
| | | | | | | | | In the Visual Script editor, the override icon is not very clear what it's intended purpose is for. Currently, its tooltip just says: "Function:" which is the same tooltip as the "add new function" icon. To resolve this issue, this PR adds descriptive tooltips to the following icons in the Visual Script editor: -Override Function -Add Function -Add Variable -Add Signal
* Changed some code showed in LGTM and Coverageqarmin2019-07-201-1/+0
|
* Use base `Color()` constructors instead of `Color::html()`Hugo Locurcio2019-07-081-74/+74
| | | | | This results in slightly smaller binaries (-17 KB for an editor binary) as no strings need to be allocated.
* Merge pull request #29333 from hbina/fix_double_click_when_no_edited_funcRémi Verschelde2019-07-011-1/+3
|\ | | | | fixed an issue with double clicking available node in VisualScriptEditor
| * fixed an issue where there is no edited function but user double clicks on ↵hbina0852019-05-301-1/+3
| | | | | | | | an available functions
* | Made use of semicolons more consitent, fixed formattingJohnJLight2019-06-191-2/+2
| |
* | Merge pull request #28766 from pgoral/editor_validation_issueRémi Verschelde2019-06-171-0/+3
|\ \ | | | | | | Changing method signature in other class in not recognized in working…
| * | Changing method signature in other class in not recognized in working class ↵Goral2019-05-291-0/+3
| | | | | | | | | | | | in typed GDScript #28685
* | | Merge pull request #29262 from DarknessCatt/issue-27476Rémi Verschelde2019-06-111-0/+3
|\ \ \ | |_|/ |/| | Automatically add new line to scripts
| * | Automatically add new line to scriptsMatheus Lima Cunha2019-05-291-0/+3
| | |
* | | Remove some redundant linesJames Buck2019-05-291-3/+0
| | |
* | | Merge pull request #28836 from Jummit/vs-fix-default-call-modesRémi Verschelde2019-05-241-69/+74
|\ \ \ | | | | | | | | Change the call mode to self when adding nodes via the node search
| * | | change call mode to self when adding nodes with node searchJummit2019-05-121-69/+74
| | |/ | |/| | | | | | | fixes #28614
* | | Merge pull request #28980 from Jummit/vs-rename-change-base-typeRémi Verschelde2019-05-241-1/+1
|\ \ \ | | | | | | | | Fix VisualScript Change Base Type title
| * | | Fix VisualScript Change Base Type titleJummit2019-05-181-1/+1
| |/ /
* / / Fix history state setup in VisualScriptSwarnim Arun2019-05-201-1/+1
|/ /
* | VS: Give the generic search the current base type.K. S. Ernest (iFire) Lee2019-05-011-4/+3
| |
* | Merge pull request #27707 from Calinou/tweak-message-wordingRémi Verschelde2019-04-301-1/+1
|\ \ | | | | | | Improve wording of various messages and make casing more consistent
| * | Improve wording of various messages and make casing more consistentHugo Locurcio2019-04-211-1/+1
| | | | | | | | | | | | | | | This also adds the number of selected projects to the confirmation dialog that appears before removing projects.
* | | Added a marker in text_edit that tells which row is executing.Rikhardur Bjarni Einarsson2019-04-231-0/+8
| | |
* | | position nodes from the generic search at the cursorJummit2019-04-181-0/+2
| |/ |/|
* | Properly handle different border widths in rounded corner aaPouleyKetchoupp2019-03-191-3/+3
|/ | | | Note: removed (unused and not implemented) support for multiple border colors
* Make translatable some undo/redo operations in the editorMichael Alexsander Silva Dias2019-02-211-1/+1
|
* Update copyright statements to 2019Rémi Verschelde2019-01-011-2/+2
| | | | Happy new year to the wonderful Godot community!
* Merge pull request #20609 from YeldhamDev/menu_hover_explicitRémi Verschelde2018-12-091-0/+1
|\ | | | | Make opening menus with the same parent on mouse focus explicit
| * Make opening menus with the same parent on mouse focus explicitMichael Alexsander Silva Dias2018-11-201-0/+1
| |
* | add undeclared function params of VisualScriptEditor::_selected_method , ↵Liangdi2018-11-211-1/+1
|/ | | | Fixes #23475
* Merge pull request #23537 from fire/visualscript-virtual-_function-crash-23536Rémi Verschelde2018-11-061-1/+1
|\ | | | | Fix creating a visual script virtual function after minimizing crashes #23536 and misc vs warning
| * On virtual method creation cancel don't do anything.K. S. Ernest (iFire) Lee2018-11-051-1/+1
| |
* | Fix empty script interface crash on tscn load.K. S. Ernest (iFire) Lee2018-11-061-1/+2
|/ | | | Add fail conditions to protect the visual script function and also fix the initiating cause.
* Updates visualscript editor when theme is changedChaosus2018-10-281-5/+13
|
* Merge pull request #22771 from guilhermefelipecgs/fix_inspector_previewRémi Verschelde2018-10-071-1/+1
|\ | | | | Fix inspector preview
| * Fix inspector previewGuilherme Felipe2018-10-071-1/+1
| | | | | | | | Remove script preview to inspector dock
* | Remove redundant "== false" codeAaron Franke2018-10-061-2/+2
| | | | | | | | | | | | Some of this code has been re-organized. f
* | Remove redundant "== true" codeAaron Franke2018-10-061-2/+2
|/ | | If it can be compared to a boolean, it can be evaluated as one in-place.
* Fix warnings about set but unused variables [-Wunused-but-set-variable]Rémi Verschelde2018-09-271-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following GCC 5 warnings: ``` drivers/gles2/rasterizer_canvas_gles2.cpp:814:8: warning: variable 'rt_size' set but not used [-Wunused-but-set-variable] drivers/gles2/rasterizer_scene_gles2.cpp:2270:11: warning: variable 'vp_height' set but not used [-Wunused-but-set-variable] drivers/gles2/rasterizer_scene_gles2.cpp:2673:22: warning: variable 'e' set but not used [-Wunused-but-set-variable] drivers/gles2/rasterizer_scene_gles2.cpp:715:7: warning: variable 'no_cull' set but not used [-Wunused-but-set-variable] drivers/gles2/shader_gles2.cpp:693:14: warning: variable 'cc' set but not used [-Wunused-but-set-variable] drivers/gles3/rasterizer_canvas_gles3.cpp:1226:8: warning: variable 'rt_size' set but not used [-Wunused-but-set-variable] drivers/gles3/rasterizer_scene_gles3.cpp:3039:10: warning: variable 'contrib' set but not used [-Wunused-but-set-variable] drivers/gles3/rasterizer_scene_gles3.cpp:4504:32: warning: variable 'vp_height' set but not used [-Wunused-but-set-variable] editor/editor_inspector.cpp:272:9: warning: variable 'guide_color' set but not used [-Wunused-but-set-variable] editor/editor_themes.cpp:1067:14: warning: variable 'alpha3' set but not used [-Wunused-but-set-variable] editor/editor_themes.cpp:263:8: warning: variable 'script_bg_color' set but not used [-Wunused-but-set-variable] editor/plugins/collision_shape_2d_editor_plugin.cpp:326:11: warning: variable 'cpoint' set but not used [-Wunused-but-set-variable] editor/plugins/mesh_editor_plugin.cpp:72:9: warning: variable 'size' set but not used [-Wunused-but-set-variable] editor/plugins/shader_editor_plugin.cpp:471:12: warning: variable 'mpos' set but not used [-Wunused-but-set-variable] editor/plugins/shader_editor_plugin.cpp:89:8: warning: variable 'basetype_color' set but not used [-Wunused-but-set-variable] editor/plugins/shader_editor_plugin.cpp:90:8: warning: variable 'type_color' set but not used [-Wunused-but-set-variable] editor/plugins/shader_editor_plugin.cpp:92:8: warning: variable 'string_color' set but not used [-Wunused-but-set-variable] modules/visual_script/visual_script_editor.cpp:2521:7: warning: variable 'seq_connect' set but not used [-Wunused-but-set-variable] platform/android/export/export.cpp:580:12: warning: variable 'styles_count' set but not used [-Wunused-but-set-variable] platform/android/export/export.cpp:584:12: warning: variable 'styles_offset' set but not used [-Wunused-but-set-variable] platform/osx/export/export.cpp:464:9: warning: variable 'zerr' set but not used [-Wunused-but-set-variable] scene/2d/tile_map.cpp:260:10: warning: variable 'tcenter' set but not used [-Wunused-but-set-variable] scene/3d/light.cpp:166:7: warning: variable 'editor_ok' set but not used [-Wunused-but-set-variable] scene/3d/navigation.cpp:566:11: warning: variable 'closest_navmesh' set but not used [-Wunused-but-set-variable] scene/gui/rich_text_label.cpp:869:8: warning: variable 'size' set but not used [-Wunused-but-set-variable] scene/main/viewport.cpp:705:14: warning: variable 'xform' set but not used [-Wunused-but-set-variable] scene/main/viewport.cpp:706:8: warning: variable 'ss' set but not used [-Wunused-but-set-variable] scene/main/viewport.cpp:726:14: warning: variable 'xform' set but not used [-Wunused-but-set-variable] scene/main/viewport.cpp:727:8: warning: variable 'ss' set but not used [-Wunused-but-set-variable] scene/resources/material.cpp:430:7: warning: variable 'using_world' set but not used [-Wunused-but-set-variable] servers/visual/shader_language.cpp:2026:7: warning: variable 'all_const' set but not used [-Wunused-but-set-variable] servers/visual/visual_server_scene.cpp:1383:28: warning: variable 'z_max_cam' set but not used [-Wunused-but-set-variable] ``` Also fixes two [-Wunused-value] warnings: ``` scene/gui/text_edit.cpp:4405:20: warning: statement has no effect [-Wunused-value] servers/visual/visual_server_scene.cpp:905:48: warning: value computed is not used [-Wunused-value] ``` Some of those are bugs and need further work, they are identified with `// FIXME` comments.
* Fix warnings about unused variables [-Wunused-variable]Rémi Verschelde2018-09-271-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes the following GCC 5 warnings: ``` drivers/gles2/rasterizer_scene_gles2.cpp:1139:15: warning: unused variable 'offset' [-Wunused-variable] drivers/gles2/rasterizer_scene_gles2.cpp:1205:39: warning: unused variable 'multi_mesh' [-Wunused-variable] drivers/gles2/rasterizer_storage_gles2.cpp:359:7: warning: unused variable 'srgb' [-Wunused-variable] drivers/gles2/shader_gles2.cpp:1016:45: warning: unused variable 'texture_hints' [-Wunused-variable] editor/animation_track_editor.cpp:776:9: warning: unused variable 'keys_to' [-Wunused-variable] editor/editor_inspector.cpp:273:7: warning: unused variable 'vs_height' [-Wunused-variable] editor/editor_themes.cpp:202:10: warning: unused variable 'begin_time' [-Wunused-variable] editor/editor_themes.cpp:239:10: warning: unused variable 'end_time' [-Wunused-variable] editor/plugins/animation_blend_tree_editor_plugin.cpp:726:17: warning: unused variable 'an' [-Wunused-variable] editor/plugins/script_text_editor.cpp:1278:8: warning: unused variable 'fold_state' [-Wunused-variable] main/main.cpp:132:13: warning: 'use_vsync' defined but not used [-Wunused-variable] modules/cvtt/image_compress_cvtt.cpp:231:8: warning: unused variable 'y_end' [-Wunused-variable] modules/cvtt/image_compress_cvtt.cpp:311:6: warning: unused variable 'shift' [-Wunused-variable] modules/gdscript/gdscript_editor.cpp:58:7: warning: unused variable 'th' [-Wunused-variable] modules/gridmap/grid_map.cpp:1084:6: warning: unused variable 'ofs' [-Wunused-variable] modules/theora/video_stream_theora.cpp:442:9: warning: unused variable 'tr' [-Wunused-variable] modules/visual_script/visual_script_editor.cpp:2606:6: warning: unused variable 'count' [-Wunused-variable] modules/visual_script/visual_script_editor.cpp:2829:6: warning: unused variable 'seq_count' [-Wunused-variable] modules/visual_script/visual_script_editor.cpp:2844:24: warning: unused variable 'vnode_function' [-Wunused-variable] modules/websocket/lws_peer.cpp:122:12: warning: unused variable 'peer_data' [-Wunused-variable] modules/websocket/lws_peer.cpp:135:12: warning: unused variable 'peer_data' [-Wunused-variable] modules/websocket/lws_peer.cpp:63:12: warning: unused variable 'peer_data' [-Wunused-variable] modules/websocket/lws_peer.cpp:91:12: warning: unused variable 'peer_data' [-Wunused-variable] platform/android/export/export.cpp:763:16: warning: unused variable 'node_size' [-Wunused-variable] scene/gui/rich_text_label.cpp:850:10: warning: unused variable 'x_ofs' [-Wunused-variable] scene/gui/text_edit.cpp:653:8: warning: unused variable 'tab_w' [-Wunused-variable] scene/resources/bit_mask.cpp:186:6: warning: unused variable 'i' [-Wunused-variable] scene/resources/mesh.cpp:549:20: warning: '_array_name' defined but not used [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:107:10: warning: unused variable 'v2' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:108:10: warning: unused variable 'v3' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:109:10: warning: unused variable 'v4' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:110:10: warning: unused variable 'v5' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:111:10: warning: unused variable 'v0n' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:112:10: warning: unused variable 'v1n' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:113:10: warning: unused variable 'v2n' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:114:10: warning: unused variable 'v3n' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:115:10: warning: unused variable 'v4n' [-Wunused-variable] servers/audio/audio_rb_resampler.cpp:116:10: warning: unused variable 'v5n' [-Wunused-variable] servers/visual/default_mouse_cursor.xpm:2:21: warning: 'default_mouse_cursor_xpm' defined but not used [-Wunused-variable] ```
* Fix warnings on non-static data member initializers (C++11 feature)Rémi Verschelde2018-09-271-0/+1
| | | | | | | | | | | | | | We're not formally using C++11 yet so those trigger compilation warnings (at least with GCC 5): ./main/input_default.h:122:30: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 CursorShape default_shape = CURSOR_ARROW; ^ Note: We may allow those eventually (especially for non-int static const), but most of current occurrences were inconsistent with all other classes. See also http://www.stroustrup.com/C++11FAQ.html#member-init
* Merge pull request #21982 from luzpaz/misc-typosRémi Verschelde2018-09-131-5/+5
|\ | | | | Misc. typos
| * Misc. typosluz.paz2018-09-121-5/+5
| | | | | | Found via `codespell -q 3 -I ../godot-word-whitelist.txt --skip="./thirdparty,*.po"`
* | Make core/ includes absolute, remove subfolders from include pathRémi Verschelde2018-09-121-4/+4
|/ | | | | | This allows more consistency in the manner we include core headers, where previously there would be a mix of absolute, relative and include path-dependent includes.
* Change the default variable constructor to use EditorInspector and updated ↵K. S. Ernest (iFire) Lee2018-09-081-5/+3
| | | | property hints that were changed.
* Make some debug prints verbose-only, remove othersRémi Verschelde2018-08-241-1/+0
|
* Do not use theme to set LineEdit right_iconŁukasz Rutkowski2018-08-111-1/+1
|
* Add clear text button to LineEditŁukasz Rutkowski2018-08-111-0/+1
| | | | | | | | | | - Add pressed state to clear button - Enable clear button on all inputs with search icon - Remove duplicate clear buttons - Fix rendering of icon for center and right alignments - Add clear button to more search fields - Add clear icon to default theme - Add method to control enabled state of clear button - Add property to enable clear button from inspector
* Carry property hint and hint string through to Visualscript virtual functions.K. S. Ernest (iFire) Lee2018-07-271-1/+1
|
* Merge pull request #20457 from fire/vs_generic_search_crash_and_connectingRémi Verschelde2018-07-271-14/+17
|\ | | | | Visualscript: misc generic search changes
| * Visualscript fix crash and generic search does not connect ports.K. S. Ernest (iFire) Lee2018-07-261-14/+17
| | | | | | | | * Signal change requires function changes to _selected_new_virtual_method
* | Visualscript: Don't type guess on nil.K. S. Ernest (iFire) Lee2018-07-261-7/+7
|/
* Fix Visual Script duplicate issueSwarnim Arun2018-07-261-2/+2
|