diff options
author | 风青山 <idleman@yeah.net> | 2024-01-21 12:42:21 +0800 |
---|---|---|
committer | 风青山 <idleman@yeah.net> | 2024-01-23 00:05:11 +0800 |
commit | 0c6b6fe74959bf812de8d5f7b019c621bf62df9c (patch) | |
tree | 47ab28a5212691a879e8387483b0c307fd7df2f4 | |
parent | 0bcc0e92b3f0ac57d4c4650722f347593a258572 (diff) | |
download | redot-engine-0c6b6fe74959bf812de8d5f7b019c621bf62df9c.tar.gz |
Fix issue where `set_global_position(global_position)` in `Control` resulted in a different result than `global_position`
Previously, the case where the pivot point was not at the origin was ignored.
-rw-r--r-- | scene/gui/control.cpp | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/scene/gui/control.cpp b/scene/gui/control.cpp index 349102fafb..a7be5c9af0 100644 --- a/scene/gui/control.cpp +++ b/scene/gui/control.cpp @@ -1419,13 +1419,15 @@ void Control::_set_global_position(const Point2 &p_point) { void Control::set_global_position(const Point2 &p_point, bool p_keep_offsets) { ERR_MAIN_THREAD_GUARD; - Transform2D inv; - if (data.parent_canvas_item) { - inv = data.parent_canvas_item->get_global_transform().affine_inverse(); + Transform2D global_transform_cache = get_global_transform(); + if (p_point == global_transform_cache.get_origin()) { + return; // Edge case, but avoids calculation. } - set_position(inv.xform(p_point), p_keep_offsets); + Point2 internal_position = global_transform_cache.affine_inverse().xform(p_point); + + set_position(internal_position + data.pos_cache, p_keep_offsets); } Point2 Control::get_global_position() const { |