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

Auto-completion should merge the selected proposal with the current line #23756

Closed
Zylann opened this issue Nov 16, 2018 · 0 comments · Fixed by #45393
Closed

Auto-completion should merge the selected proposal with the current line #23756

Zylann opened this issue Nov 16, 2018 · 0 comments · Fixed by #45393

Comments

@Zylann
Copy link
Contributor

Zylann commented Nov 16, 2018

Godot 3.1 alpha2 and before

Sometimes when I use auto-complete, it occurs that I'm at a position where previous code exists after my cursor. When I select a completion proposal, it just appends the result to my cursor, which causes temporary errors and I have to delete the code that followed manually. In VisualStudio, Intellisense actually merges the following code, so that you don't need to do this extra deletion.

Examples:
I renamed or moved a node, so I need to change its path in code. The following scenario then happens:
I place my cursor in the path, press Ctrl+Space to get suggestions (or just start typing a name, either way)
image
I choose the proposal, and... I end up with this.
image
I then have to correct it myself to go back to correct code...
image
It would be nice if the time gained by autocomplete didnt get lost in this^^

The same happens for member access. Let's say I want to replace the call to a function by another of a similar name. I also get auto-completion proposals:
image
But as soon as I choose one, I end up with this:
image
And you guess I have to move the cursor to correct this out manually.

WDYT?

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