这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@rolandwalker
Copy link
Contributor

Description

Bind the keys which may initiate an fzf-based reverse search to also iterating upward through the matched items, once we are in fzf mode.

This makes fzf search more closely match the keyboard muscle memory of traditional reverse incremental search (though it may look very different visually).

Like #1278 this is intended to address

This also assumes that alt-r from #1278 can initiate an fzf search.

Checklist

  • I've added this contribution to the changelog.md.
  • I've added my name to the AUTHORS file (or it's already there).

@rolandwalker rolandwalker self-assigned this Jul 19, 2025
@rolandwalker rolandwalker force-pushed the RW/bind-ctrl-r-alt-r-in-fzf-search branch from c2bc44b to 58f6ea5 Compare July 19, 2025 12:44
@rolandwalker rolandwalker requested review from amjith and lazmond3 July 19, 2025 12:47
Bind the keys which may initiate an fzf-based reverse search to also
iterating upward through the matched items, once we are in fzf mode.

This makes fzf search more closely match the keyboard muscle memory of
traditional reverse incremental search (though it may look very
different visually).

Like #1278 this is intended to address

 * #1265 (comment)

This also assumes that alt-r from #1278 can initiate an fzf search.
@rolandwalker rolandwalker force-pushed the RW/bind-ctrl-r-alt-r-in-fzf-search branch from 58f6ea5 to 9efa7d0 Compare July 19, 2025 14:45
Copy link
Contributor

@lazmond3 lazmond3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding detailed options when using fzf!

@rolandwalker rolandwalker merged commit 7363615 into main Jul 19, 2025
7 checks passed
@rolandwalker rolandwalker deleted the RW/bind-ctrl-r-alt-r-in-fzf-search branch July 19, 2025 14:49
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