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.

Sign up for GitHub

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

Jump to bottom

v:event.size is 0 in CompleteChanged event if ext_popupmenu is used #20646

Closed
zeertzjq opened this issue Oct 14, 2022 · 0 comments · Fixed by #20648
Closed

v:event.size is 0 in CompleteChanged event if ext_popupmenu is used #20646

zeertzjq opened this issue Oct 14, 2022 · 0 comments · Fixed by #20648
Labels
bug issues reporting wrong behavior completion Nvim built-in (omni)completion complexity:low Low-risk, self-contained. Do NOT ask "can I work on this", just read CONTRIBUTING.md events events, autocommands ui-extensibility UI extensibility, events, protocol

Comments

@zeertzjq
Copy link
Member

zeertzjq commented Oct 14, 2022 •
edited
Loading

Neovim version (nvim -v)

v0.9.0-dev-82-g0ef6aaa3a7

Vim (not Nvim) behaves the same?

N/A

Operating system/version

Arch Linux

Terminal name/version

N/A

$TERM environment variable

N/A

Installation

build from repo

How to reproduce the issue

  1. Run nvim with a UI with ext_popupmenu
  2. Run :let g:sizes = []
  3. Run :autocmd CompleteChanged * let g:sizes += [v:event.size]
  4. Type i<C-X><C-V><Esc>
  5. Run :echo g:sizes

Expected behavior

A list with two non-zero values is printed

Actual behavior

[0, 0] is printed
@zeertzjq zeertzjq added bug issues reporting wrong behavior ui-extensibility UI extensibility, events, protocol completion Nvim built-in (omni)completion events events, autocommands complexity:low Low-risk, self-contained. Do NOT ask "can I work on this", just read CONTRIBUTING.md labels Oct 14, 2022
@zeertzjq zeertzjq changed the title v:event.size is always 0 in CompleteChanged event if ext_popupmenu is used v:event.size is 0 in CompleteChanged event if ext_popupmenu is used Oct 14, 2022
@zeertzjq zeertzjq mentioned this issue Oct 14, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug issues reporting wrong behavior completion Nvim built-in (omni)completion complexity:low Low-risk, self-contained. Do NOT ask "can I work on this", just read CONTRIBUTING.md events events, autocommands ui-extensibility UI extensibility, events, protocol
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(completion): set pum_size even if ext_popupmenu is used zeertzjq/neovim
1 participant
@zeertzjq