summaryrefslogtreecommitdiffstats
path: root/modules/visual_script/visual_script_editor.cpp
Commit message (Collapse)AuthorAgeFilesLines
* 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
|
* Visualscript: Don't show Shift+A search box at every screen and reuse ↵K. S. Ernest (iFire) Lee2018-07-251-8/+6
| | | | | | EDIT_FIND_NODE_TYPE (ctrl-f) shortcut. The generic search shortcut is now CTRL-F.
* Merge pull request #18867 from fire/better_vx_us_rebase_02Juan Linietsky2018-07-241-141/+307
|\ | | | | Improve VisualScript UX
| * Improve VisualScript UXK. S. Ernest (iFire) Lee2018-07-241-141/+307
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Prototype faster function call ux. * Work on general search ux. * Able to create nodes from search. * Show class for variables but not methods. * Get actions search working. * Descriptions now show for both methods and properties. * Enable zooming on mouse wheel up and down. * Make the drag trigger on right mouse button. * Search now shows for action visual script nodes. * Able to search visual node names. * Search works better. * Change zooming scale to hide artifacts better. * Remove zoom changes * Select from base should check properties too like the other functions. * Seq_connect flag is needed to set sequence lines correctly. * Remove comment * Code cleanup with function names and arguments. * Use brief description for search descriptions. * Clean and fix bug with input nodes connecting with sequence lines. * Add a warning and fix some edge conditions with sequence into data lines and vice versa. * Don't search functions when pulling from a sequence node. * Don't show actions when pulling from a data line. * Set set and get properties. * Convert visual script operators to the correct type * Create a function preset finds only functions. * Singletons can now find functions. * Add shift-a for generic search. * Add brief descriptions for Visual Script nodes. * Search boxes can now filter names. * Add bigger hit zones to node connect. * For the drop zones, make all the rect2 areas the same size. * Function names in visual script node should be lower case so that search works better. * Use the convention of capitalize() for set, set, visual script nodes and methods. * Make search more general. Ignore "_" and make case-insensitive. Also made the search window smaller and remove extra info from search * Make type_cast use the connecting node's type and remove use of found variable. * Fix case where you call an instance's call function where it becomes an invalid call. * Make get_visual_node_names use a set of filters, move action creations and fix bug with sequence node connections. * Make the window bigger. * Make connect_data and connect_seq more robust. * Add icons to search items. * Add vs constructors in shift-a menu. * Operators, builtins and constructors show type name. Fix several problems with port connections. * In shift-a mode search everything. * Code cleanup * Work on autocompleting the type. * Use type guess in action creation. * Check if type hint string exists in object variables when generating the visual script search. * Add the hint to SceneTree. * Add original type detection. * Make type casting great again. This puts the type casted base type as the data output type string hint. * Pass the type in a VisualScriptFunctionCall too. * Set the base type correctly in VisualScriptFunctionGet and VisualScriptFunctionSet using hint string. * Make sure the instance is passed in VisualScriptPropertySet. * Restore search on the node's type. * Remove dependencies from graph_edit. * Remove dependencies from property_selector and name the class visual_script_property_selector. * Extract hot zones into a function. * Move hot_zones constants into default theme. * Bigger capture zones. * Clean messy port_grab_distance variables. * Remove RMB functionality. * Remove memory leak on showing visual script descriptions. * Read the port_grab_distance constants on enter tree and theme changed.
* | Allow opening and editing of any utf_8 file in script editorPaulb232018-07-221-13/+12
|/
* Fix visual script editor interfaceK. S. Ernest (iFire) Lee2018-07-101-0/+1
|
* Merge pull request #18516 from PJB3005/18-04-30-visual-script-title-barsMax Hilbrunner2018-05-261-7/+26
|\ | | | | Move things into the title bars of Visual Script nodes.
| * Move things into the title bars of Visual Script nodes.Pieter-Jan Briers2018-04-301-7/+26
| | | | | | | | | | | | This is an attempt to make VS slightly nicer to use. WiP.
* | -New inspector.Juan Linietsky2018-05-151-1/+1
|/ | | | | | | -Changed UI resizing code, gained huge amount of speed. -Reorganized timer sync to clean up behavior (sorry forgot commit this before) -
* Abstracted the syntax highlighter from text editPaulb232018-04-021-0/+6
|
* Merge pull request #15379 from ianb96/delete_key_fixRémi Verschelde2018-02-141-6/+38
|\ | | | | Filesystem and Visual Script Members delete key fix
| * Filesystem and Visual Script Members keyboard shortcuts fixIan2018-01-061-6/+38
| |
* | Make sure PropertyHint matches in the VisualScript editor.K. S. Ernest (iFire) Lee2018-01-151-1/+2
| |
* | Add missing copyright headers and fix formattingRémi Verschelde2018-01-051-0/+1
|/ | | | | | Using `misc/scripts/fix_headers.py` on all Godot files. Some missing header guards were added, and the header inclusion order was fixed in the Bullet module.
* Update copyright statements to 2018Rémi Verschelde2018-01-011-2/+2
| | | | Happy new year to the wonderful Godot community!
* Merge pull request #14754 from willnationsdev/dictionary-copyRémi Verschelde2017-12-171-3/+3
|\ | | | | Added 'duplicate' function for Dictionary in C++ and API.
| * Dictionary::copy -> ::duplicateWill Nations2017-12-171-3/+3
| |
* | Now every variant type has its icon.Daniel J. Ramirez2017-12-171-8/+8
| |
* | General use Variant type icons, plus other icon updates.Daniel J. Ramirez2017-12-171-52/+52
|/
* Made Vector::ptrw explicit for writing, compiler was sometimes using the ↵Juan Linietsky2017-11-251-1/+1
| | | | | | wrong function, leading to unnecesary copy on writes and reduced performance.
* Merge pull request #11933 from cxong/masterJuan Linietsky2017-11-211-3/+3
|\ | | | | Use "Command" instead of "Meta" for macOS (#1619)
| * Use "Command" for OSX in translation strings (#1619)Cong2017-10-131-3/+3
| |
* | Rename Rect3 to AABB.Ferenc Arn2017-11-171-2/+2
| | | | | | | | Fixes #12973.
* | Create API to add and remove VisualScript custom nodesGeorge Marques2017-11-151-0/+43
| | | | | | | | | | | | | | This makes a VisualScriptEditor singleton, which gives plugins the ability to register their own custom nodes. Those will be available for insertion in the Visual Script editor, under the "Custom Nodes" category.
* | Properly fix node path selection in input ports, closes #12152Juan Linietsky2017-11-121-0/+17
| |
* | (VS) Add space after "=" sign in variable listhomer6662017-10-171-1/+1
|/
* VS now supports white themes.Daniel J. Ramirez2017-10-021-51/+105
|