search

linux-cultist / venv-selector.nvim

Allows selection of python virtual environment from within neovim
MIT License
388 stars 41 forks source link

Lazy load telescope viewer #51

Closed neolooong closed 11 months ago

neolooong commented 11 months ago

When venv-selector is loaded, telescope is also loaded. Even though I didn't set the telescope as a dependency in the lazy spec, and didn't use "VenvSelect" yet. I think It is common practice to set telescope.nvim to lazy load.

So, I moved all the "requires telescope" statements into functions so that telescope.nvim can be loaded when it is actually needed.

Also, I think top-to-bottom search result is better if the input box is at the top.

linux-cultist commented 11 months ago

Thank you very much!

And yes, we can switch to "top-to-bottom" search results. If users hate it, we can always add an option later to let them choose.

Lets get this once merged.