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

Restore greyed out area after jump to 1st match after a timeout #62

Closed
mikehaertl opened this issue Oct 8, 2021 · 3 comments
Closed
Labels
API enhancement New feature or request

Comments

@mikehaertl
Copy link

First: thanks for this awesome plugin! I really start to get the hang of it.

But after some weeks of use I found one thing that could maybe be tweaked. It's about this feature (from the README):

the cursor automatically jumps to the first match by default; pressing any other key than a group-switch or a target label exits the plugin now

After typing 2 chars I often find, that I'm already where I wanted to go. But now the area is still greyed out. All I wanted to do is go to that new location, e.g. to reason about some piece of code. But my brain feels disturbed a lot by the still greyed out area. I now feel a disruption because I have think about "How can I get rid of that greyed out area?". I could press a key without any binding (ESC is to far away. On my german keyboard ä may be a good candidate). But it still requires me to press yet another key - for no real action.

I wonder if we couldn't just get a (configurable) timeout: After the cursor jumped to the first match, we could wait x seconds and then exit the plugin and restore the greyed out area.

Does this make sense? Or is there an even better alternative that I'm missing?

@ggandor
Copy link
Owner

ggandor commented Oct 9, 2021

Sure, this is a sensible idea. I think I even tried it once when developing the plugin, and for some inexplicable reason, found it annoying, but my memory is a bit foggy on that... I will implement this again, maybe push up a branch so that you folks can test it out.

(I personally remap caps lock to esc on the OS level, like many Vimmers, so I've never found this too problematic, reaching to the left with the pinky is such an automatic movement...)

@ggandor ggandor added the enhancement New feature or request label Oct 9, 2021
@ggandor ggandor added the API label Nov 5, 2021
@ggandor
Copy link
Owner

ggandor commented Nov 5, 2021

Implemented by be9e1b4. (Wrong reference in the commit message. To err is human.) EDIT: And here I messed up again, naturally :)

@ggandor ggandor closed this as completed Nov 5, 2021
@mikehaertl
Copy link
Author

Seems to work fine, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants