Skip to content
New issue

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

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

Already on GitHub? # to your account

Prevent completion menu after navigation in insert mode #5803

Closed

Conversation

p1gp1g
Copy link
Contributor

@p1gp1g p1gp1g commented Feb 3, 2023

This is a replacement for #4144 , after the comment from @the-mikedavis :

It would be better to be able [...] to prevent the completion menu from opening at all after any navigation in insert mode

Related issues: #2027 #2581.

The auto-completion menu now pops if you are in insert mode AND you have inserted something, as soon as you move again or do a ^C, you need to insert something again. This behavior is similar to what other IDE do (like Android Studio)

@pascalkuthe pascalkuthe added A-helix-term Area: Helix term improvements S-waiting-on-review Status: Awaiting review from a maintainer. labels Feb 6, 2023
@gibbz00 gibbz00 mentioned this pull request Feb 22, 2023
5 tasks
@p1gp1g
Copy link
Contributor Author

p1gp1g commented Apr 4, 2023

I just wanted to kindly follow up on my pull request and mention that I've been using it for the past 2 months and the modification has made the completion natural and instinctive. Is there anything else I can do to help move the process along?

@7flash
Copy link

7flash commented Aug 14, 2023

Yes please merge, it solves very annoying issue corrupting my code all the time

@pascalkuthe
Copy link
Member

we won't merge adhoc fixes like this. We are working on revamping thway completions are triggered more generally: #6910 (comment)

@pascalkuthe pascalkuthe mentioned this pull request Aug 20, 2023
@p1gp1g p1gp1g force-pushed the prevent_completion_navigation branch from 6053b56 to 8d9b92f Compare November 8, 2023 08:28
@p1gp1g
Copy link
Contributor Author

p1gp1g commented Nov 8, 2023

I have rebased the branch, for users using that patch.

I don't know if it could be merged until the revamping is done (then reverted) ?

@pascalkuthe
Copy link
Member

This is fixed by #8021 which is a pretty high priority now. I don't see us merging another solution before that PR

@p1gp1g
Copy link
Contributor Author

p1gp1g commented Nov 8, 2023

I see, I should test that PR then

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A-helix-term Area: Helix term improvements S-waiting-on-review Status: Awaiting review from a maintainer.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants