Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Transform3D::looking_at() missing use_model_front #1644

Closed
sash-rc opened this issue Nov 18, 2024 · 2 comments · Fixed by #1715
Closed

Transform3D::looking_at() missing use_model_front #1644

sash-rc opened this issue Nov 18, 2024 · 2 comments · Fixed by #1715
Labels
bug This has been identified as a bug
Milestone
4.x

Comments

@sash-rc
Copy link

sash-rc commented Nov 18, 2024 •
edited
Loading

Godot version

>= 4.3

godot-cpp version

>= 4.3

System information

Any

Issue description

Transform3D::looking_at() doesn't have use_model_front parameter, as in GDScript counterpart

Steps to reproduce

n/a

Minimal reproduction project

n/a
@dsnopek
Copy link
Collaborator

dsnopek commented Nov 18, 2024

Thanks!

We recently fixed this for Basis in PR #1625

Looks like we need to do the same for Transform3D

@dsnopek dsnopek added the bug This has been identified as a bug label Nov 18, 2024
@dsnopek dsnopek added this to the 4.x milestone Nov 18, 2024
@sash-rc
Copy link
Author

sash-rc commented Nov 18, 2024

@dsnopek, thanks, and I forgot to mention, probably the same fix should be applied to the Transform3D::set_look_at() too

@dsnopek dsnopek mentioned this issue Feb 21, 2025
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
bug This has been identified as a bug
Projects
None yet
Milestone
4.x
Development

Successfully merging a pull request may close this issue.

Synchronize most shared variant code with Godot 4.4 dsnopek/godot-cpp
2 participants
@dsnopek @sash-rc