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

Combine home row letters as target labels #103

Closed
petobens opened this issue May 6, 2014 · 2 comments
Closed

Combine home row letters as target labels #103

petobens opened this issue May 6, 2014 · 2 comments

Comments

@petobens
Copy link

petobens commented May 6, 2014

If I use g:sneak#target_labels default value things work fine (as you can see in the first image below). However, if I set

let g:sneak#target_labels = "asdfghjkl"

then there are missing on-screen target locations (as seen in the second image). It would be nice if sneak could combine whatever is set in g:sneak#target_labels (in my case the home row) to show all the on-screen locations i.e use labels such as ds or gl to jump to targets.
screenshot-06-05-2014_15-29-42
screenshot-06-05-2014_15-30-01

@justinmk
Copy link
Owner

justinmk commented May 6, 2014

Press <tab> to label the next N matches.

Sneak doesn't show multi-letter labels because:

  • makes adjacent labels difficult to discern
  • adds cognitive overhead

The idea is to optimize for the common case. Since you specify 2 characters in advance, in general (the common case) you should not have too many matches on-screen.

The adjacent label problem can be solved with a bunch of code, but I don't think the benefits are worth it (EasyMotion tried/tries many approaches to this). I'm more interested in solving the real problem, which is text with a high degree of uniformity (eg, "aaaaaaaaaaaaaaaaaaaaaaa").

Some ideas are discussed in #88, plus I have another approach I have been testing.

Thanks for your interest!

@justinmk justinmk closed this as completed May 6, 2014
@petobens
Copy link
Author

petobens commented May 6, 2014

Thanks for the thorough explanation. Excellent plugin! :)

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

No branches or pull requests

2 participants