Languageclient-neovim: [Enhancement] Context menu

[kristijanhusak]

LSP has a lot of options, and mapping all those or calling them manually is a bit annoying.
We could create a context menu that will give user a selection.
Something like this with a built in confirm function:

Or inputlist

Maybe the second option is better because there's a lot of options, and first one goes horizontally, where the second one goes vertically.

If you are interested I'll create a PR.

[autozimu]

Interesting idea! Please go for it.

On Wed, May 23, 2018 at 02:09 Kristijan Husak [e-mail]
wrote:

LSP has a lot of options, and mapping all those or calling them manually
is a bit annoying.
We could create a context menu that will give user a selection.
Something like this with a built in confirm function:
[image: screenshot]
https://camo.githubusercontent.com/fe171a62c86da8dad7ee971c5ae6d7bca0f16878/68747470733a2f2f692e696d6775722e636f6d2f6b7652664d57652e706e67

Or inputlist
[image: screenshot_inputlist]
https://camo.githubusercontent.com/2eec2642177472d26c3cf7bd83744e756643459f/68747470733a2f2f692e696d6775722e636f6d2f7a6a566e6f4d4c2e706e67

Maybe the second option is better because there's a lot of options, and
first one goes horizontally, where the second one goes vertically.

If you are interested I'll create a PR.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/autozimu/LanguageClient-neovim/issues/442, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYt75TtoH7M1lMUBqUfvOK-YSVC8cRaks5t1SdfgaJpZM4UKDyj
.

[DerWeh]

Sounds like a great idea. Maybe LanguageClient-neovim could just provide a Denite menu with its commands, as it already integrated with Denite.

[KillTheMule]

That would restrict the feature to denite users, though.

[Anrock]

How about also provide a Denite menu?

[KillTheMule]

Sounds like a plan :)

[kristijanhusak]

If we would support denite, then i guess we should support fzf too, like for everything else. This escalated quickly :D

[Anrock]

Yeah. Denite and fzf support could be moved to separate issues, though.

[kristijanhusak]

I'll try to create shared solution for all 3.

[autozimu]

Merged in #446.