-
Notifications
You must be signed in to change notification settings - Fork 28
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
Control timeout of instant jump/instant repeat labels separately #150
Comments
No, that is not entirely true. It is as simple and consistent as possible: if there are labels on the screen, the On the other hand, it is an intentional feature that the labels usually remain active even after The problem is, I'm actually considering removing the timeout option entirely. It is a pretty useless gimmick actually: you most probably want to do something after a jump; the only exception I could think of is if one has some very big |
...and indeed it is the case! Thanks for bringing this to my attention, I'll look into it ASAP. Still, what about the rest (1. keeping the labels available as a worthwhile feature 2. do we need timeout at all)? |
After experimenting a little bit more, I actually found the thing that was most disorienting to me was
For highlights, I definitely like those timing out (both for instant repeat and for Sorry if I'm unclear! I've generally found lightspeed to behave intuitively, but it seems like you built some complex concepts to accomplish that. I definitely had to spend a decent amount of time reading the manual to try to map what I'm seeing onto all those concepts. Lightspeed is definitely my favorite easymotion-style plugin though. Thanks for considering my feedback! |
I discovered your leap.nvim plugin, and tried it. I actually find it less disorienting, even with So maybe leap.nvim is just more what I'm looking for, and my thoughts on lightspeed are less relevant? Either way, kudos for your great plugins. |
Currently, all uses of 2-character search use the same timeout (
exit_after_idle_msecs.labeled
). However, I would like the timeout to behave differently when an instant jump is triggered. Ultimately, the config I would like to use is for normal labeled jumps to have no timeout (so I have time to process the labels that have been selected), but I would like instant jumps to behave like 1 character search (e.g. the labels disappear quickly).My general reasoning is that for a normal 2-character search, I'm not done until after I've selected the final label, and once a final label is selected all labels disappear. When using an instant repeat, there will continue to be labels even after my cursor has reached its final destination, and I would like those to disappear automatically after a short timeout.
This is especially confusing for me when an instant jump is triggered, since it feels like lightspeed has exited already, but there are still phantom labels in case I choose to do an instant repeat after.
I'm not sure exactly what fits best with this plugin's philosophy, but one of these two things would definitely match my intuition:
exit_after_idle_msecs.instant
timeout, that I can set to some small valueexit_after_idle_msecs.unlabeled
for instant jumps.The text was updated successfully, but these errors were encountered: