Skip to content

Conversation

prabirshrestha
Copy link
Owner

let g:lsp_loaded = 1

let g:lsp_auto_enable = get(g:, 'lsp_auto_enable', 1)
let g:lsp_hover_balloon_eval = get(g:, 'lsp_hover_balloon_eval', has('balloon_eval'))
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to support this scenario. let g:lsp_hover_ui = 'echo,preview,balloon'

@mattn
Copy link
Collaborator

mattn commented Dec 21, 2018

Please wait merge until fixed a bug that vim call API for balloon with free-ed pointer.

@prabirshrestha
Copy link
Owner Author

Can you add a reference/link to the vim bug/discussion here for future reference.

@mattn
Copy link
Collaborator

mattn commented Dec 25, 2018

I confirmed two problems about tooltip.

  1. balloon_show does not show utf-8 strings vim/vim#3708
  2. Broken text

About No2, I don't confirm how to reproduce.

@mattn
Copy link
Collaborator

mattn commented Dec 28, 2018

vim/vim#3708 was merged.

@mattn
Copy link
Collaborator

mattn commented Dec 28, 2018

One more vim/vim#3725

@mattn
Copy link
Collaborator

mattn commented Jan 8, 2019

Now both patches are merged into Vim.

@mattn
Copy link
Collaborator

mattn commented Jan 11, 2019

You should want to enable balloonevalterm?

@prabirshrestha
Copy link
Owner Author

We can check this in once we have the event queue and diff changes in. Would help reduce talking too much to the server.

@prabirshrestha
Copy link
Owner Author

closing in favor of the new vim/neovim popup feature which also works in terminal and is more powerful than balloon. #395

@prabirshrestha prabirshrestha deleted the balloon_eval_hover branch June 15, 2019 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants