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

Feature: Fuzzy find dropbar menu entries #65

Closed

Conversation

theofabilous
Copy link
Contributor

fzf

Adds a fuzzy finding feature for dropbar menus using telescope-fzf-native.

Similar to #58, but uses mostly pure lua code, whereas #58 had lots of ffi calls, structs, etc. which made the changes much more complex.

Todo

  • Mouse hover support (mousemoveevent)
  • More/better configuration options
  • Properly restore cursor column in menu when exiting fuzzy finding mode:
    • When entering and leaving insert mode, the cursor moves backwards by one char, so when leaving fzf and returning to menu, the cursor moves back, which can be hacked around by moving the cursor forward, but this has some weird behaviour with quick navigation enabled

Things I'm considering but may not implement:

  • Recursive searching, i.e. search for the entries in the current menu AND their children, recursively

@willothy
Copy link
Collaborator

willothy commented Sep 1, 2023

Any updates on this / plans to finish it? Would be very nice to have. I'd be happy to contribute if that would help :)

@willothy willothy mentioned this pull request Sep 2, 2023
7 tasks
@theofabilous
Copy link
Contributor Author

Any updates on this / plans to finish it? Would be very nice to have. I'd be happy to contribute if that would help :)

Hey sorry for the late reply @willothy. Thanks so much for your work on this! I've been a little busy and wasn't able to keep working on this at the time so I'm really glad you picked this up. I will close this PR given your work in #77.

@willothy
Copy link
Collaborator

willothy commented Sep 9, 2023

No worries at all! Thanks for your work as well, this was really a great idea.

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.

2 participants