-
Notifications
You must be signed in to change notification settings - Fork 84
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
Toggle search highlight no longer works #88
Comments
This was a deliberate change, in response to a request for a way to clear the current search pattern. In retrospect, this feature should probably not have been overloaded onto the ESC-u command. I think it would have been better to leave ESC-u as a toggle and add a new command, perhaps ESC-U, to clear the search pattern. Although I'm somewhat loathe to change the functionality yet again, I'm leaning toward changing ESC-u back to the old behavior as you request and adding a new command to clear the search pattern. |
All right. Thanks for the explanation! |
Fixed in 7b1787e. |
I use to toggle highlighting of search results back and forth with ESC-u, but since commit 5651293 this no longer works. I would like to understand the rationale behind this change, if this was indeed the intended behaviour. (And if not, consider this a plain old bug report.)
The text was updated successfully, but these errors were encountered: