Skip to content
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

feat: use vim.ui.select for 'pick one prompts` #572

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

oysandvik94
Copy link

Does this: #500

Part of this: #358

I guess multiselect depends on this:

neovim/neovim#18161

@Pistonight
Copy link

This looks like exactly what I need. What's blocking this from being merged?

@Jouo
Copy link

Jouo commented Apr 30, 2024

@mfussenegger What are the chances of merging this?

@mfussenegger
Copy link
Owner

Not sure, I haven't made up my mind yet how to deal with this.

Some of the functionality assumes that the buffer remains visible but vim.ui.select is often implemented as floating window that hides the buffer content.
That makes the disambiguation logic for organizing imports less useful. Currently it jumps and highlights to the expressions for which you're prompted to choose an import.

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

Successfully merging this pull request may close these issues.

4 participants