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

Skeleton3D Insert Key button is impossible to read #8957

Closed
hunterbk opened this issue Feb 14, 2024 · 3 comments · Fixed by godotengine/godot#88386
Closed

Skeleton3D Insert Key button is impossible to read #8957

hunterbk opened this issue Feb 14, 2024 · 3 comments · Fixed by godotengine/godot#88386
Labels

Comments

@hunterbk
Copy link

Your Godot version:
Godot 4.2.stable

Issue description:
At the top ribbon when you are animating a Skeleton3D, there is a button to insert all bone positions/rotations as keys to an existing track, but it just about gave me a stroke reading it. Moreover, I tried to fix it myself, but I apparently can't, but I noticed that this is literally the key for all translations, which means all translations should probably be checked to make sure that they're translated properly, because the original was hardly understandable. I'm not going to fix it myself because I'm not sure how the translation keys work and I don't want to cause any issues with translations.

URL to the documentation page:
https://github.com/godotengine/godot/blob/e5a3fd1e53ad4e6efc1fd17deded8f4ce0973d10/editor/plugins/skeleton_3d_editor_plugin.cpp#L790

https://github.com/search?q=repo%3Agodotengine%2Fgodot+already+exist+track.&type=code&p=1

I suggest something like "Insert key of existing bone poses into an existing track." and also you might wanna look into the "Insert Key (All Bones) part too because it's hard to understand what the difference is between that and the "Insert Key (Existing Tracks) button.

@skyace65
Copy link
Contributor

Just so you're aware this is an issue with the engine, not the documentation. For something like this you'd want to open a Issue in the engine repository. That aside this is so bad I just opened a PR to fix this.

@hunterbk
Copy link
Author

Just so you're aware this is an issue with the engine, not the documentation. For something like this you'd want to open a Issue in the engine repository. That aside this is so bad I just opened a PR to fix this.

Got it, I think last time I used the normal engine repository and was told to come here for properties when you hover over something in the engine, it's tough to tell imo, even more so with this. I also opened another issue with the button itself being broken, but that's on the normal repo. I appreciate it, should I close this issue?

@skyace65
Copy link
Contributor

Ok I think I see where the confusion came from. By properties that person was probably referring to node properties specifically. Those issues should be opened here because node properties are part of the class reference for nodes. In this case this isn't a property, it's a tooltip for a editor button. You can leave this issue open.

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

Successfully merging a pull request may close this issue.

2 participants