evil-search-highlight-persist icon indicating copy to clipboard operation
evil-search-highlight-persist copied to clipboard

When `evil-search-module` is `evil-search` then `SPC s c` doesn't work

Open tuhdo opened this issue 10 years ago • 6 comments
trafficstars

Original issue here: https://github.com/syl20bnr/spacemacs/issues/1326

You have quite many users using your package now.

tuhdo avatar Apr 22 '15 15:04 tuhdo

I'm an emacs newbie, what should "SPC s c" do? If I disable evil-search and do "SPC s" emacs just says "SPC s is not defined".

juanjux avatar Apr 23 '15 08:04 juanjux

evil-search-highlight-persist-remove-all is bound to SPC s c, so it is expected to clear all highlighting. However, if you set evil-search-module to 'evil-search then the highlighting clear doesn't work anymore.

tuhdo avatar Apr 23 '15 08:04 tuhdo

Mmmm, by default evil-search-highlight-persist-remove-all is bound to C-x SPC. I've tried to set evil-search-module to evil-search and C-x SPC still works for me.

juanjux avatar Apr 23 '15 08:04 juanjux

@juanjux Out of interest did you restart Emacs after changing evil-search-module? I found that changing the setting alone did not have any effect until Emacs was restarted. I guess Evil has to be reloaded or something?

When using evil-search the highlighting appears to be different from the normal isearch highlighting, which is usually how you can tell which one is currently in use. (And you can also press <up> to go through previous search history from the search minibuffer.)

jonathanj avatar Apr 23 '15 12:04 jonathanj

Conversation about this issue has recently come up again and I noticed there was no reply to my suggestion as to how to reproduce it. (Which has effectively helped others to reproduce it.)

Do you have any better idea why changing the search module might break this functionality?

jonathanj avatar Nov 14 '15 05:11 jonathanj

Hi! The problem is that I'm not using Emacs anymore, I went back to pure (Neo)Vim, so if you want to fork an maintain this go ahead!

juanjux avatar Nov 27 '15 08:11 juanjux