-
-
Notifications
You must be signed in to change notification settings - Fork 21k
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 editor settings for autocompletion with NodePath and StringName #66481
Conversation
d09451a
to
9a1f33b
Compare
9a1f33b
to
1cbbaf4
Compare
Messed up the rebase....... Edit: Ok I think I didn't nuke my PR while trying to fix it. Should be good again? |
6d75275
to
d2aba36
Compare
d2aba36
to
5da70c6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok, except 1 description.
5da70c6
to
74b8fa3
Compare
May I poke in? This PR has been good for a whole year and it's only missing @Paulb23 's approval and a conflict merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implementation seems fine. I'm not sure the feature will actually be used by anyone though, as those literals don't seem to make much difference to performance. But it makes sense a priori.
This feature was suggested by Calinou in another PR, hence my poking. |
I'd like to ping @Paulb23 one more time for good measure to make sure there are no issues with the PR as is (minus the fact that it needs to be rebased again) |
If this feature is desired, apart from a rebase, updated changes look okay to me. |
74b8fa3
to
178305f
Compare
178305f
to
de8b6f7
Compare
Add editor settings for autocompletion with NodePath and StringName
06abc86
Thanks! |
Closes godotengine/godot-proposals#5496
This is built into the autocompletion algorithm – if you're in a function and autocompletion gives a string, it checks whether the editor setting is on. If it is, it adds the literals
&
or^
as prefixes, based on if the argument is a NodePath or a StringName. Works in any situation, no adaptation is needed to specific functions.autocomplete.stonks.mp4