Skip to content

Support skim as an alternative to fzf #879

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
pickfire opened this issue Sep 14, 2019 · 5 comments
Closed

Support skim as an alternative to fzf #879

pickfire opened this issue Sep 14, 2019 · 5 comments

Comments

@pickfire
Copy link

Is your feature request related to a problem? Please describe.

Skim is a fuzzy finder similar to fzf. It can be integrated to Vim using the Skim.vim plugin. It would be nice to make Vista.vim compliant with skim to not have to install fzf in addition.
By the way, skim is also written in rust, related PR solved in liuchengxu/vista.vim#129

Describe the solution you'd like

skim is an alternative to FZF, and skim.vim and fzf.vim does not differs much from the look.

Describe alternatives you've considered

skim.vim is the alternative to fzf.vim

Additional context

@ghost
Copy link

ghost commented Sep 23, 2019

I created a shim function [1] to call skim instead of fzf in vim. It should 'just work'. The PR hasent been merged thou.
[1]: skim-rs/skim.vim#7

@pickfire
Copy link
Author

Looks interesting how short that patch is, I don't get how it works though. I know skim does provide both fzf#wrap() and fzf#run().

@ghost
Copy link

ghost commented Sep 24, 2019

I know skim does provide both fzf#wrap() and fzf#run().

No, it doesnt. Thats what the patch adds, and it just reuses the arguments with skim#run/wrap

edit: obviously it wouldnt work well if fzf-vim is also present.

benbarbour added a commit to benbarbour/skim.vim that referenced this issue May 26, 2020
Copied from skim-rs#7, but moved to
the requested file. I'm doing it since the author of skim-rs#7 seems to have
deleted thier account.

Hoping to close autozimu/LanguageClient-neovim#879
@martskins
Copy link
Collaborator

@pickfire now that #1059 has been merged to dev, I think you should be able to configure skim as your selection UI. Let me know if that fixes this for you.

@pickfire
Copy link
Author

@pickfire now that #1059 has been merged to dev, I think you should be able to configure skim as your selection UI. Let me know if that fixes this for you.

Looks like I was already using skim before that since fzf stuff are already present even though I didn't install it, most likely because skim is a drop-in replace for fzf, it didn't even change the naming and the plugin still call it fzf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants