omnisharp-vim icon indicating copy to clipboard operation
omnisharp-vim copied to clipboard

Support denite as selector UI

Open afk-mario opened this issue 7 years ago • 7 comments

Shougo new plugin replacing Unite is Denite, from the Readme:

Unite.vim is anything.el like interface for Vim. But the implementation is ugly and very slow. Denite.nvim resolves unite.vim problems. It is 10 times faster than unite.vim.

It would be nice to be able to choose it as the selector UI for Omnisharp

afk-mario avatar Feb 13 '17 17:02 afk-mario

Just bumping this. If one of the maintainers can lead me in the right direction, I'd be willing to look into putting together a PR for this.

tulsileathers avatar Feb 21 '18 13:02 tulsileathers

@nosami ?

tulsileathers avatar Feb 21 '18 13:02 tulsileathers

Sorry, I haven't worked in C# or vim for a long time now.

@thinca is the maintainer

nosami avatar Feb 21 '18 13:02 nosami

Sorry just went off who had the most commits. Thanks for pinging the right person!

tulsileathers avatar Feb 21 '18 13:02 tulsileathers

PR is welcome! Feel free to contact me if you have any questions!

thinca avatar Feb 24 '18 08:02 thinca

@tulsileathers I'm not a maintainer but I've been tinkering with OmniSharp-vim for a while. I've also just recently been updating some of the fzf/ctrlp/unite integration code to update the plugin to use 'v2' of the roslyn /codeactions functionality which is far superior (see #342).

As I said in that PR, I don't have any experience with unite or denite. If you do have experience with unite, perhaps a good place to start would be to take a look at the PR and see if you can get the existing unite functionality to use 'v2' instead of 'v1', as I've done for fzf and ctrlp?

Adding denite as a 4th option after that shouldn't be too complicated.

nickspoons avatar Apr 03 '18 22:04 nickspoons

@tulsileathers are you still interested in having a look at this? Previous PRs are now merged, and I worked out the unite stuff so that's up-to-date too. If you want to have a go at adding denite now, that'd be great!

nickspoons avatar Apr 18 '18 02:04 nickspoons