Age | Commit message (Collapse) | Author |
|
Fixes reported logically dead codes by Coverity
* image.cpp: Doesn't really need any modification. But to remove the bug
report then we have to move the MAX call away from the for loop
statement.
* rasterizer_gles3.cpp: Removes unnecessary elif condition since it is
checked earlier in the function
* collada.cpp: If stamement never reached due to macro ERR_CONTINUE does
the same.
* navigation_mesh.cpp: Variables should always be null - however, also
checked for the very same condition in their function call. Leaving this
for review (whether the function call is necessary or not)
* path_editor_plugin.cpp: If cancel is true, then it should restore the
edited value to the original provided.
http://docs.godotengine.org/en/3.0/classes/class_editorspatialgizmo.html#class-editorspatialgizmo-commit-handle
* spatial_editor_gizmos.cpp: the very condition of i >= 3 is
predetermined in the if case right before it. Thus case 1 is always '1'
and case 2 is always '-1'
* grid_map_editor.cpp: Same as above in spatial_editor_gizmos.cpp
* voxel_light_baker.cpp: Same as above in spatial_editor_gizmos.cpp
* visual_server.cpp: Same as above in spatial_editor_gizmos.cpp
* visual_script_expression.cpp: char '-' is already true in the switch
case mechanism. Thus it can never reach to default case.
* particles.cpp: Case 'PARAM_MAX' is unreachable due to index checking
right before the switch execution.
* shader_language.cpp: Invalid index is handled in switch default case.
`type < TYPE_FLOAT && type > TYPE_VEC4` -> `(type < TYPE_FLOAT || type > TYPE_VEC4`)
Fixes the "always false problem" in TODO comment.
|
|
|
|
Prevent visibility notification from being called twice in object creation
|
|
Add popup_closed signal for ColorPickerButton
|
|
Avoid converting Quat to Euler angles when not necessary.
|
|
Fix index out of size error on Image
|
|
Fix #18164
|
|
Fix #18177
|
|
Fix #18229
|
|
Also ensure that get_scale doesn't arbitrarlity change the signs of scales, ensuring that the combination of get_rotation and get_scale gives the correct basis.
Added various missing functions and constructors.
Should close #17968.
|
|
ItemList needs to check against the number of items available when the user moves the selection via "ui_right" action.
|
|
Fixes #17688
|
|
Remove the rect surrounding 2D nodes in 2D editor when it's not pertinent
|
|
|
|
|
|
|
|
|
|
Closes #18043
|
|
LineEdit placeholder alignment, content margins, and overflow bugs
|
|
Add support for radio-looking items with icon
|
|
Make Button not emit press when container is scrolled
|
|
Made Node.print_tree() print prettily
|
|
LineEdit doesn't correctly uses style margins nor use placeholders
width correctly, causing multiple rendering bugs.
|
|
Fix of RigidBody's configuration warning for Z axis
|
|
Drag to Rearrange Editor Docks
|
|
Letting users of `PopupMenu` use them. `OptionButton` was one of those interested and is updated in this commit.
Fixes #18063.
|
|
Fixed colour regions and added local colour region cache
|
|
This fixes the problem described in #13996 in a proper way.
This also adds "deadzone" property to ScrollContainer. It can be used
on mobile, where taps are not as precise as mouse clicks. Player could
slightly move their finger when tapping, in which case we still want
the button to be pressed rather than the container to be scrolled.
|
|
|
|
|
|
expose Tileset TileMode to GDScript
|
|
|
|
|
|
Add flip switch to SurfaceTool.generate_normals
|
|
Fix Sprite3D's incorrect behavior when using AtlasTextures.
|
|
Added GROW_DIRECTION_BOTH for controls
|
|
Added `get_playing_speed` method to AnimationPlayer, returning the actual playing speed of the animation being played
|
|
[RFC] MultiplayerAPI refactor
|
|
Duplicate Arrays and Dictionaries when instancing scene in editor
|
|
Fix converting to tileset crashes Godot if existing file is not tileset
|
|
not sure why.
Fixes #17524, fixes #17523.
|
|
|
|
Removed unnecessary newlines from tscn
|
|
Particles 2D and 3D - fix hue variation not working
|
|
Radio buttons in menus
|
|
Fix `RichTextLabel` column size growing beyond its calculated `max_width`
|
|
Add tint property to TextureProgress
|
|
Improve popup menus usability
|
|
|
|
It seems that popups were intended to "grab" the mouse click that triggered them, but their intent was being lost. This commit does the necessary changes to let it happen and updates items that were trying to get advantage of it, because the semantics of `Control::grab_click_focus()` have changed a bit. Namely, it must be called **before** showing the modal.
This allows to popup a menu and activate an item in it in a single click-point-release cycle, instead of having to click once to open the menu and once more to pick an item.
This ability is extended even to context menus activated with the RMB (or any other mouse button, for that matter). The editor benefits from this in the context menu of the tree dock, which has been patched to opt-in for this feature.
This improves UX a bit by saving unnecessary clicks.
From now on, `PopupMenu` always grabs the click and also invalidates the first button release unless the mouse has moved (that's what `set_invalidate_click_until_motion()` was doing and now it's removed), so there is no longer the need of doing both things at every point a pop-up menu is shown.
|