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: improve autojump #1997

Merged
merged 6 commits into from
Feb 10, 2025
Merged

feat: improve autojump #1997

merged 6 commits into from
Feb 10, 2025

Conversation

psicho2000
Copy link
Contributor

In case of z.lua with fzf, the experience can be improved: currently, we need to potentially input twice. First for the initial query, second for filtering. We can improve this in two ways:

  1. Delegate the selection process completely to fzf: pipe all dirs into fzf and let it select the entry after the last slash
  2. Remove the weightings - they are not needed while selecting.

@psicho2000 psicho2000 force-pushed the feat/improve-autojump branch 2 times, most recently from fe663b5 to fa96d11 Compare February 8, 2025 18:39
@psicho2000 psicho2000 force-pushed the feat/improve-autojump branch from fa96d11 to 5c5105e Compare February 8, 2025 18:45
@psicho2000 psicho2000 force-pushed the feat/improve-autojump branch from 5c5105e to 9fc8288 Compare February 8, 2025 18:49
@psicho2000 psicho2000 force-pushed the feat/improve-autojump branch from 9fc8288 to 3f54351 Compare February 8, 2025 18:54
plugins/autojump Outdated Show resolved Hide resolved
plugins/autojump Outdated Show resolved Hide resolved
@psicho2000 psicho2000 force-pushed the feat/improve-autojump branch from c5bb359 to e0ed6c8 Compare February 9, 2025 10:13
@psicho2000 psicho2000 requested review from jarun and N-R-K February 9, 2025 10:13
plugins/autojump Show resolved Hide resolved
@psicho2000 psicho2000 requested a review from jarun February 9, 2025 14:22
@jarun jarun merged commit 1976cad into jarun:master Feb 10, 2025
8 checks passed
@jarun
Copy link
Owner

jarun commented Feb 10, 2025

Thank you!

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.

3 participants