search

echasnovski / mini.nvim

Library of 40+ independent Lua modules improving overall Neovim (version 0.8 and higher) experience with minimal effort
MIT License
4.45k stars 171 forks source link

Feature: add option jump_to_single_result to lsp picker #978

Closed abeldekat closed 2 weeks ago

abeldekat commented 2 weeks ago

Contributing guidelines

Module(s)

mini.pick, mini.extra

Description

Recently I started using mini.pick in my config. Thank you for this great plugin!

Regarding the lsp picker, I miss the ability to jump directly to a single result, bypassing the picker. This is standard behavior in telescope, and can be configured in fzf-lua by supplying jump_to_single_result is true.

I frequently use goto definition. For now, fzf-lua provides the following lsp pickers in my config: "definition", "reference", "implementation", "type_definition".

Would it be possible to add this behavior to mini.pick?

echasnovski commented 2 weeks ago

Thanks for the suggestion!

This is, of course, possible. However, I don't think this is a good idea. My reasoning is that starting a picker should always result into showing a picker (as stated in #809 with a bit more thoughts in this comment).

Closing as not planned.

abeldekat commented 2 weeks ago

Oops... I was just writing an apology for this duplicate issue...)