summaryrefslogtreecommitdiffstats
path: root/editor
Commit message (Collapse)AuthorAgeFilesLines
* Split `Geometry` singleton into `Geometry2D` and `Geometry3D`Andrii Doroshenko (Xrayez)2020-05-2711-35/+44
| | | | Extra `_2d` suffixes are removed from 2D methods accoringly.
* Add `custom_modules` build option to compile external user modulesAndrii Doroshenko (Xrayez)2020-05-252-15/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds ability to include external, user-defined C++ modules to be compiled as part of Godot via `custom_modules` build option which can be passed to `scons`. ``` scons platform=x11 tools=yes custom_modules="../project/modules" ``` Features: - detects all available modules under `custom_modules` directory the same way as it does for built-in modules (not recursive); - works with both relative and absolute paths on the filesystem; - multiple search paths can be specified as a comma-separated list. Module custom documentation and editor icons collection and generation process is adapted to work with absolute paths needed by such modules. Also fixed doctool bug mixing absolute and relative paths respectively. Implementation details: - `env.module_list` is a dictionary now, which holds both module name as key and either a relative or absolute path to a module as a value. - `methods.detect_modules` is run twice: once for built-in modules, and second for external modules, all combined later. - `methods.detect_modules` was not doing what it says on the tin. It is split into `detect_modules` which collects a list of available modules and `write_modules` which generates `register_types` sources for each. - whether a module is built-in or external is distinguished by relative or absolute paths respectively. `custom_modules` scons converter ensures that the path is absolute even if relative path is supplied, including expanding user paths and symbolic links. - treats the parent directory as if it was Godot's base directory, so that there's no need to change include paths in cases where custom modules are included as dependencies in other modules.
* Fix build after merge of #37235Rémi Verschelde2020-05-251-2/+3
| | | | It used APIs that were changed after the PR was last rebased.
* Merge pull request #37235 from Calinou/improve-editor-feature-profilesRémi Verschelde2020-05-252-5/+33
|\ | | | | Improve the editor feature profiles UX
| * Improve the editor feature profiles UXHugo Locurcio2020-03-222-5/+33
| | | | | | | | | | | | | | | | | | | | | | - Add an help message when no profile is selected. - This replaces the class/property trees which are now hidden when no profile is selected. - Display `(none)` as the current profile when no profile is currently active. - Make the newly created/imported profile the current if it's the first profile to be added to the list. - Make more strings localizable.
* | Reverse mouse wheel in animation track editorTomasz Chabora2020-05-251-2/+2
| |
* | Fix moving 2D node with mouse after using arrow keysDominik 'dreamsComeTrue' Jasiński2020-05-211-1/+1
| | | | | | | | Fixes: #38894
* | DocData: Fix sorting of arguments and constantsRémi Verschelde2020-05-201-6/+12
| | | | | | | | | | | | | | The missing `operator<` definitions caused `Vector::sort()` to fail sorting those alphabetically by name on Windows (not sure why Linux isn't affected, I guess GCC/Clang are cleverer and use the operator from the first struct member).
* | Merge pull request #38876 from paulloz/fix-import-params-errorRémi Verschelde2020-05-201-1/+3
|\ \ | | | | | | Fix error with 'params' when import dock doesn't define any parameter
| * | check the params section exist before erasing it in import_dockPaul Joannon2020-05-201-1/+3
| | | | | | | | | | | | related #38864
* | | Keep "lock" metadata when changing a Node's type, if applicableMichael Alexsander2020-05-191-0/+12
|/ /
* | Merge pull request #38800 from YeldhamDev/icons_completeRémi Verschelde2020-05-182-0/+2
|\ \ | | | | | | Add more missing node icons
| * | Add more missing node iconsMichael Alexsander2020-05-172-0/+2
| | |
* | | Merge pull request #38804 from m4gr3d/android_plugin_config_masterRémi Verschelde2020-05-174-3/+66
|\ \ \ | |/ / |/| | Implementation of the Godot Android Plugin configuration file
| * | Implementation of the Godot Android Plugin configuration fileFredia Huya-Kouadio2020-05-174-3/+66
| | |
* | | Merge pull request #38695 from dreamsComeTrue/node-swap-order-argumentsRémi Verschelde2020-05-171-1/+2
|\ \ \ | |/ / |/| | Replace 'add_child_below_node' with 'add_sibling' in Node
| * | Replace 'add_child_below_node' with 'add_sibling' in NodeDominik 'dreamsComeTrue' Jasiński2020-05-151-1/+2
| | | | | | | | | | | | Fixes: #19642
* | | Merge pull request #38451 from eduardonunesp/feature/command-comma-preferencesRémi Verschelde2020-05-161-1/+4
|\ \ \ | | | | | | | | Using command + comma on macOS as default shortcut for editor settings
| * | | Using command + comma on macOS as default shortcut for editor settingsEduardo Nunes Pereira2020-05-111-1/+4
| | | |
* | | | Merge pull request #38764 from YeldhamDev/icons_missingRémi Verschelde2020-05-159-2/+6
|\ \ \ \ | |_|/ / |/| | | Add more missing icons
| * | | Add more missing iconsMichael Alexsander2020-05-159-2/+6
| | | |
* | | | Hide editor_spin_slider grabber when closing Editor's windowsDominik 'dreamsComeTrue' Jasiński2020-05-151-0/+1
|/ / / | | | | | | | | | Fixes: #38740
* | | Style: Enforce braces around if blocks and loopsRémi Verschelde2020-05-14140-3140/+5920
| | | | | | | | | | | | | | | Using clang-tidy's `readability-braces-around-statements`. https://clang.llvm.org/extra/clang-tidy/checks/readability-braces-around-statements.html
* | | Style: Enforce separation line between function definitionsRémi Verschelde2020-05-1470-0/+261
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I couldn't find a tool that enforces it, so I went the manual route: ``` find -name "thirdparty" -prune \ -o -name "*.cpp" -o -name "*.h" -o -name "*.m" -o -name "*.mm" \ -o -name "*.glsl" > files perl -0777 -pi -e 's/\n}\n([^#])/\n}\n\n\1/g' $(cat files) misc/scripts/fix_style.sh -c ``` This adds a newline after all `}` on the first column, unless they are followed by `#` (typically `#endif`). This leads to having lots of places with two lines between function/class definitions, but clang-format then fixes it as we enforce max one line of separation. This doesn't fix potential occurrences of function definitions which are indented (e.g. for a helper class defined in a .cpp), but it's better than nothing. Also can't be made to run easily on CI/hooks so we'll have to be careful with new code. Part of #33027.
* | | Style: clang-format: Disable KeepEmptyLinesAtTheStartOfBlocksRémi Verschelde2020-05-14279-7683/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Which means that reduz' beloved style which we all became used to will now be changed automatically to remove the first empty line. This makes us lean closer to 1TBS (the one true brace style) instead of hybridating it with some Allman-inspired spacing. There's still the case of braces around single-statement blocks that needs to be addressed (but clang-format can't help with that, but clang-tidy may if we agree about it). Part of #33027.
* | | Style: Fix missing/invalid copyright headersRémi Verschelde2020-05-142-2/+2
| | |
* | | Enforce use of bool literals instead of integersRémi Verschelde2020-05-149-13/+13
| | | | | | | | | | | | | | | Using clang-tidy's `modernize-use-bool-literals`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-bool-literals.html
* | | Modernize remaining uses of 0/NULL instead of nullptr (C++11)Rémi Verschelde2020-05-141-1/+1
| | | | | | | | | | | | | | | Using clang-tidy's `modernize-use-nullptr`. https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-nullptr.html
* | | Port member initialization from constructor to declaration (C++11)Rémi Verschelde2020-05-1425-330/+173
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using `clang-tidy`'s `modernize-use-default-member-init` check and manual review of the changes, and some extra manual changes that `clang-tidy` failed to do. Also went manually through all of `core` to find occurrences that `clang-tidy` couldn't handle, especially all initializations done in a constructor without using initializer lists.
* | | Merge pull request #38637 from Calinou/editor-rename-revert-scene-bindRémi Verschelde2020-05-132-5/+6
|\ \ \ | | | | | | | | Rename the editor action "Revert Scene" to "Reload Saved Scene"
| * | | Rename the editor action "Revert Scene" to "Reload Saved Scene"Hugo Locurcio2020-05-122-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This option can be used to workaround various issues with stuff not reloading properly when changes are made. The option was renamed to clarify the fact that it actually reloads the scene saved on the filesystem.
* | | | Update game controller enums.Marcel Admiraal2020-05-131-36/+47
| | | |
* | | | Support multiple debug protocols.Fabio Alessandrelli2020-05-1212-37/+75
| | | |
* | | | Update the editor icons README to remove outdated informationHugo Locurcio2020-05-121-10/+5
| | | | | | | | | | | | | | | | This closes #38684.
* | | | Tweak the error message displayd when a post-import script failsHugo Locurcio2020-05-111-1/+3
| | | | | | | | | | | | | | | | See #38662.
* | | | thirdparty: Cleanup after #38386, document provenance and copyrightRémi Verschelde2020-05-113-3/+3
| | | | | | | | | | | | | | | | Also renamed `delaunay.h` to `delaunay_2d.h` to match the class name.
* | | | Merge pull request #38386 from reduz/new-lightmapperRémi Verschelde2020-05-1116-347/+972
|\ \ \ \ | |_|/ / |/| | | New GPU lightmapper
| * | | New lightmapperJuan Linietsky2020-05-1016-347/+972
| |/ / | | | | | | | | | | | | | | | | | | -Added LocalVector (needed it) -Added stb_rect_pack (It's pretty cool, we could probably use it for other stuff too) -Fixes and changes all around the place -Added library for 128 bits fixed point (required for Delaunay3D)
* | | Merge pull request #38592 from Calinou/debugger-error-warning-iconRémi Verschelde2020-05-113-6/+11
|\ \ \ | | | | | | | | Use a different icon for the debugger tab with both warnings and errors
| * | | Use a different icon for the debugger tab with both warnings and errorsHugo Locurcio2020-05-093-6/+11
| | | | | | | | | | | | | | | | | | | | This makes it possible to see if both errors and warnings were pushed without having to open the tab.
* | | | Merge pull request #38594 from Calinou/rename-clear-script-actionRémi Verschelde2020-05-112-24/+24
|\ \ \ \ | | | | | | | | | | Rename the Clear Script editor action to Detach Script
| * | | | Rename the Clear Script editor action to Detach ScriptHugo Locurcio2020-05-092-24/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes it more obvious that the script won't be modified in any way. See comments in #27813.
* | | | | Merge pull request #38631 from Calinou/editor-log-selection-color-themeRémi Verschelde2020-05-111-0/+2
|\ \ \ \ \ | |_|_|/ / |/| | | | Tweak the editor log selection color to match the current editor theme
| * | | | Tweak the editor log selection color to match the current editor themeHugo Locurcio2020-05-101-0/+2
| | | | | | | | | | | | | | | | | | | | This overrides the default blue color.
* | | | | Revert "Renamed plane's d to distance"Rémi Verschelde2020-05-105-15/+15
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ec7b481170dcd6a7b4cf0e6c1221e204ff7945f3. This was wrong, `d` is not a distance but the `d` constant in the parametric equation `ax + by + cz = d` describing the plane.
* | | | Style: clang-format: Disable AllowShortCaseLabelsOnASingleLineRémi Verschelde2020-05-1016-116/+298
| | | | | | | | | | | | | | | | Part of #33027.
* | | | Style: clang-format: Disable AllowShortIfStatementsOnASingleLineRémi Verschelde2020-05-1017-61/+122
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of #33027, also discussed in #29848. Enforcing the use of brackets even on single line statements would be preferred, but `clang-format` doesn't have this functionality yet.
* | | | Style: Add missing copyright headersRémi Verschelde2020-05-106-0/+180
| | | |
* | | | Renamed plane's d to distanceMarcus Elg2020-05-105-15/+15
|/ / /
* / / Rename Lineshapes d to distanceMarcus Elg2020-05-091-6/+6
|/ /