junegunn / fzf.vim

fzf :heart: vim
MIT License
9.55k stars 583 forks source link

Add clp support #1427

Open jpe90 opened 1 year ago

jpe90 commented 1 year ago

This PR adds support for clp as an alternative to bat for file previewing.

In this implementation, clp is always used for file previewing if it's found. It assumes if the program is installed, the user would want to use it for file previewing, since that's its only function.

Alternatively a configuration option could be used to let users opt-in to clp more explicitly. Let me know if you would prefer doing it that way, or taking some other approach.

jpe90 commented 1 year ago

@junegunn not currently, but that sounds like a good idea to me. I'll work on it.

jpe90 commented 1 year ago

@junegunn I noticed that bat line numbers don't seem to have background highlighting when they're on a highlighted line.

Just to clarify, were you referring to the grey coloring of the number line text?

Screen Shot 2022-09-22 at 2 21 43 PM
junegunn commented 1 year ago

Just to clarify, were you referring to the grey coloring of the number line text?

Yes. With clp SOME_FILE | nl, we don't get highlighted line numbers.