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

Avy should merge/replace with the classic '/' search command. #7210

Closed
fausterjames opened this issue Sep 28, 2016 · 2 comments
Closed

Avy should merge/replace with the classic '/' search command. #7210

fausterjames opened this issue Sep 28, 2016 · 2 comments
Labels
- Mailling list - stale marked as a stale issue/pr (usually by a bot)

Comments

@fausterjames
Copy link

The search command should be replaced by avy, save the letter n and N to keep the native next and last motion and appart from that just replace every first char in matches by a letter that you just have to hit to jump to it. Basically merge all the commands avy-goto-char, word, line into just the search matches.

Maybe it would just be sufficient to add a little thing to the search function : if search string is 1 char long then avy-goto-char, and av-goto-word otherwise

For the search results that are off screen it would just react like the search command.

@TheBB
Copy link
Collaborator

TheBB commented Sep 28, 2016

Disagree, these two things are too different.

For the search results that are off screen it would just react like the search command.

Yeah this doesn't sound great, I don't want my searching to behave completely different just based on what I'm looking for is visible or not.

I'm not convinced at all that your suggested implementation even works.

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid!

@github-actions github-actions bot added the stale marked as a stale issue/pr (usually by a bot) label Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Mailling list - stale marked as a stale issue/pr (usually by a bot)
Projects
Mailing list
  
Pending merge
Development

No branches or pull requests

2 participants