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

Improves menu selection mechanics (regular vim) #75

Merged

Conversation

dsabsay
Copy link

@dsabsay dsabsay commented Sep 14, 2020

This fixes the issue described in #58. When opening the popup window, the cursor is now moved to the first link item in the list. Additionally, when j/k are pressed, any non-link menu items are skipped over.

I've implemented and tested with regular terminal vim 8.2. I'm not sure if Neovim had this problem or not.

* Non-link items are skipped over when pressing j/k
* First link is selected automatically when popup is opened
@airblade
Copy link

Solves the problem for me (Vim 8.2) – thanks @dsabsay!

@tristandunn
Copy link

Works for me in vim 8.2.1750 as well and honestly makes the plug-in a 100% more useful. Would be great to get this merged.

Thanks @dsabsay!

@pechorin pechorin changed the base branch from master to menu-selection-prerelease December 7, 2020 22:47
@pechorin pechorin merged commit 8c8e55c into pechorin:menu-selection-prerelease Dec 7, 2020
@pechorin
Copy link
Owner

pechorin commented Dec 7, 2020

Sorry for delay, thanks for work, but this doesn't work in this scenario:

Снимок экрана 2020-12-08 в 02 03 53

I can't select [10 more]

Also it's stop working on long lists (more then ~30 items)

Снимок экрана 2020-12-08 в 02 05 34

Sorry for big delay, i think now i could do approvement without delays (1-2 days max)

@pechorin pechorin mentioned this pull request Dec 7, 2020
@pechorin
Copy link
Owner

pechorin commented Dec 7, 2020

Also, cursor starting position fixed in #79

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.

None yet

4 participants