Use an UI to override/implement methods.

[DasOhmoff]

Problem Statement

Hey 👋, thank you for your nice work!

Right now, when I try to implement methods, using the command line to select them is pretty uncomfortable, and also not very nice looking:

It would be nice to have some UI to be able to do that instead.

Ideas or possible solutions

One could use f.e telescope to be able to select that. Telescope offers the ability to select the items: I am just demonstrating the Idea, of course selecting multiple code actions and applying them does not work with LSPs.

Thank you very much for your support :)

[mfussenegger]

Improving the UI sounds good, but I don't want nvim-jdtls to have a dependency on telescope and would rather have an abstraction that allows users to plug-in different implementations.

A first step would be to change jdtls.ui.pick_many to be async. Then it could be overridden with a telescope specific implementation. (See also this PR in neovim that would go into this direction: https://github.com/neovim/neovim/pull/18161)

[DasOhmoff]

Nice. I like this. I did not mean to add a telescope dependency anyway, I was just showing that the concept of the UI would improve things.

So is there a plan or so in regards to this issue?

[mfussenegger]

With https://github.com/mfussenegger/nvim-jdtls/pull/360 all jdtls.ui.pick_many calls are now happening inside a coroutine, so it would be possible to do something like:

require('jdtls.ui').pick_many = function(items, prompt, label_f, opts)
  local co = coroutine.running()
  call_other_async_version(..., function(result)
    coroutine.resume(co, result)
  end)
  return coroutine.yield()
end)

Changing the default implementation is still on my todo

[DasOhmoff]

Ah, perfekt. Thanks! Waiting for the default too 👀