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

Add drag and drop from inspector and to script #3256

Closed
nosdregamon opened this issue Sep 5, 2021 · 2 comments
Closed

Add drag and drop from inspector and to script #3256

nosdregamon opened this issue Sep 5, 2021 · 2 comments
Milestone

Comments

@nosdregamon
Copy link

nosdregamon commented Sep 5, 2021

Describe the project you are working on

I'm editing a 3d AnimationTree and once again I notice, that the properties aren't displayed as they are actually used in the script. Usually I just need to lower case a property, but with the long chains of animationTree-properties it get's a bit out of hand....

Describe the problem or limitation you are having in your project

To use AnimationTree in the script I have to use $AnimationTree.set("parameters/idleWalkRun/blend_position", myValue), but "parameters/idleWalkRun/blend_position" is displayed as:
Parameters
-> Idle Walk Run
-> Blend Position

So I either have to guess whether to use Upper/Lower-Case vs. underscore, or I have to select the property and hover over it.

Describe the feature / enhancement and how it helps to overcome the problem or limitation

If I'm already hovering over the property, I would love to drag the inspector property fields into the script editor, where they are needed.

Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams

Kinda like I can drag and drop files from the FileSystem-Dock into the script and get the needed string.
Same could be useful for nearly every property.

If this enhancement will not be used often, can it be worked around with a few lines of script?

I don't think that's possible with a simple script/addon. But I'm not very experienced in this field.

Is there a reason why this should be core and not an add-on in the asset library?

I don't think that's possible with a simple script/addon. But I'm not very experienced in this field.

@YuriSizov
Copy link
Contributor

This is already implemented by godotengine/godot#51628 and godotengine/godot#51629

@YuriSizov YuriSizov added this to the 3.x milestone Sep 5, 2021
@nosdregamon
Copy link
Author

Thank you!

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

No branches or pull requests

2 participants