Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

LookAtModifier3D documentation data has typos #99532

Closed
crei0 opened this issue Nov 22, 2024 · 0 comments · Fixed by #99533
Closed

LookAtModifier3D documentation data has typos #99532

crei0 opened this issue Nov 22, 2024 · 0 comments · Fixed by #99533

Comments

@crei0
Copy link
Contributor

crei0 commented Nov 22, 2024

Tested versions

  • Reproducible in: Godot v4.4.dev5 snapshot

System information

Godot v4.4.dev5 - Pop!_OS 22.04 LTS on X11 - X11 display driver, Multi-window, 1 monitor - Vulkan (Forward+) - dedicated AMD Radeon RX 6750 XT (RADV NAVI22) - AMD Ryzen 5 5600 6-Core Processor (12 threads)

Issue description

After the introduction of Implement LookAtModifier3D, the newly created documentation data (doc/classes/LookAtModifier3D.xml) for the LookAtModifier3D has some typos where it uses the name SkeletonModifier3D instead of the correct LookAtModifier3D

This is visible at the documentation page at https://docs.godotengine.org/en/latest/classes/class_lookatmodifier3d.html

Location 1:
The SkeletonModifier3D rotates a bone to look a target.
it should say
The LookAtModifier3D rotates a bone to look a target.

Location 2:
This SkeletonModifier3D rotates a bone to look a target. [...]
it should say
This LookAtModifier3D rotates a bone to look a target. [...]

And on Create a new node window on the tooltip and on the description section
image

Steps to reproduce

Documentation page:

  1. Go to https://docs.godotengine.org/en/latest/classes/class_lookatmodifier3d.html

Inside the Godot's editor:

  1. Open the Create a new node window
  2. Select the LookAtModifier3D node

Minimal reproduction project (MRP)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants