nvim-tinygit icon indicating copy to clipboard operation
nvim-tinygit copied to clipboard

Feature Request: Support for pickers other than `telescope`

Open 231tr0n opened this issue 10 months ago • 2 comments

Feature Requested

Hi @chrisgrieser! Is it possible to add support to use mini.pick for interactive staging. Also the same for mini.notify in place of nvim-notify.

Relevant Screenshot

No response

Checklist

  • [x] The feature would be useful to more users than just me.
  • [x] I have read the Non-Goals section of the README.

231tr0n avatar Jan 19 '25 08:01 231tr0n

In theory yes, but the implementation of interactive staging is a few hundred lines of code. And writing that multiple times for each picker plugin (fzf-lua, mini-picker, snacks-picker, ...) is simply not sustainable, sorry.

chrisgrieser avatar Jan 19 '25 09:01 chrisgrieser

Update due to the 1.0 release:

  • nvim-notify is no longer a dependency. You can now use any notification plugin.
  • telescope is only a hard dependency for interactive staging. For all other cases where a picker is used, the plugin still uses telescope, but due to the tinygit-rewrite, it should be relatively straightforward to add support for other pickers.

If anyone is willing to make PRs for that, check out this file: https://github.com/chrisgrieser/nvim-tinygit/blob/main/lua/tinygit/shared/selector.lua

chrisgrieser avatar Mar 16 '25 14:03 chrisgrieser

okay, in the recent update, I went for using vim.ui.select for most things, so this plugin works with any picker.

Only exception interactive staging, which simply requires telescope because that part is too complicated for vim.ui.select, thus still requiring telescope.

chrisgrieser avatar May 09 '25 11:05 chrisgrieser

So the telescope plugin is still a hard dependency?

wvffle avatar May 13 '25 16:05 wvffle

For interactive staging, yes. For everything else, you only need a picker that supports vim.ui.select.

chrisgrieser avatar May 13 '25 16:05 chrisgrieser