mfussenegger / nvim-jdtls

Extensions for the built-in LSP support in Neovim for eclipse.jdt.ls
GNU General Public License v3.0
1k stars 62 forks source link

feat: use vim.ui.select for 'pick one prompts` #572

Open oysandvik94 opened 9 months ago

oysandvik94 commented 9 months ago

Does this: #500

Part of this: #358

I guess multiselect depends on this:

https://github.com/neovim/neovim/pull/18161

Pistonight commented 6 months ago

This looks like exactly what I need. What's blocking this from being merged?

Jouo commented 3 months ago

@mfussenegger What are the chances of merging this?

mfussenegger commented 2 months ago

Not sure, I haven't made up my mind yet how to deal with this.

Some of the functionality assumes that the buffer remains visible but vim.ui.select is often implemented as floating window that hides the buffer content. That makes the disambiguation logic for organizing imports less useful. Currently it jumps and highlights to the expressions for which you're prompted to choose an import.