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

Find home for evil search highlight persist #1017

Closed
quicknir opened this issue Jan 24, 2018 · 2 comments
Closed

Find home for evil search highlight persist #1017

quicknir opened this issue Jan 24, 2018 · 2 comments

Comments

@quicknir
Copy link

There's been a recent mess around wiki packages being removed. One of the things that this highlighted is a small set of packages that despite having a fair number of users, are totally unmaintained (and thus, nobody was paying attention to their dependency on wiki packages): melpa/melpa#5008 (comment).

One of those packages is evil-search-highlight-persist: https://github.com/juanjux/evil-search-highlight-persist. Since this package is providing functionality that is widely used and part of vim out of the box, it's unmaintained and only 218 lines of elisp, can this simply be merged into evil? We have to figure out what will happen wrt the highlight requirement so maybe this cannot happen instantly, but hopefully it can be merged in when that issue is dealt with (or perhaps there is now an alternative that can be used to highlight).

@wasamasa
Copy link
Member

No need, if you customize evil-search-module to evil-search, you get persistent highlighting (which you can turn off with :noh[lsearch]) and a search that's closer to Vim generally.

That aside, the package did already find a new maintainer: juanjux/evil-search-highlight-persist#19. Please take it up with them if you still feel strongly about this. Personally, I'd just rewrite the code to not make use of highlight.el.

@quicknir
Copy link
Author

quicknir commented Jan 24, 2018

Ah, thanks for the info, I didn't realize this was implemented in evil already, thank you. That said, it seems like the evil version does not support highlighting in all buffers, which vim does. So shouldn't existing code providing that feature be integrated? #805.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants