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

fuzzy completion acts as if 'noinsert' is enabled #15009

Closed
habamax opened this issue Jun 15, 2024 · 6 comments
Closed

fuzzy completion acts as if 'noinsert' is enabled #15009

habamax opened this issue Jun 15, 2024 · 6 comments
Labels

Comments

@habamax
Copy link
Contributor

habamax commented Jun 15, 2024

Steps to reproduce

  1. vim -Nu NONE
  2. add text hello helion hero
  3. :set completeopt=menu,fuzzy
  4. in a new line type he<C-n>

Expected behaviour

hello should be inserted and selected in popup menu.

Instead hello is selected only.

image

Basically it acts as if noinsert is in completeopt for non-fuzzy completion.

Version of Vim

9.1.487

Environment

debian12, bash

Logs and stack traces

No response

@habamax habamax added the bug label Jun 15, 2024
@habamax
Copy link
Contributor Author

habamax commented Jun 15, 2024

ping @glepnir

@habamax
Copy link
Contributor Author

habamax commented Jun 15, 2024

if this is intended then it should be documented, probably the same way as the longest

	   noinsert Do not insert any text for a match until the user selects
		    a match from the menu. Only works in combination with
		    "menu" or "menuone". No effect if "longest" is present.

@glepnir
Copy link
Member

glepnir commented Jun 15, 2024

#14976 (comment)

@glepnir
Copy link
Member

glepnir commented Jun 15, 2024

it already fixed in that pr. can split that code to a single pr i think

@habamax
Copy link
Contributor Author

habamax commented Jun 15, 2024

great! no need to split pr.

@habamax habamax closed this as completed Jun 15, 2024
@habamax
Copy link
Contributor Author

habamax commented Jun 15, 2024

hmm, looking at that PR I can't see it targets 'noinsert' but I might have overlooked it.

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