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

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

Closed
jonathanj opened this issue Apr 22, 2015 · 15 comments
Closed

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

jonathanj opened this issue Apr 22, 2015 · 15 comments
Labels
- Bug tracker - Evil stale marked as a stale issue/pr (usually by a bot)

Comments

@jonathanj
Copy link

It would be nice if SPC s c worked for evil-search mode too, I prefer evil-search since it's a lot like Vim's search rather than Emacs' isearch.

For some reason search highlights also look different when evil-search is the selected search module, I don't understand why.

@tuhdo
Copy link
Contributor

tuhdo commented Apr 22, 2015

Indeed when setting evil-search-module to evil-search, it can no longer clear highlighting or activate highlighting. You should report upstream.

@tuhdo
Copy link
Contributor

tuhdo commented Apr 22, 2015

@jonathanj this is unrelated to this issue but related to the issue with searching with hgrep buffer earlier. Actually you can use these key bindings to easily navigate the buffer and it won't conflict with evil-mode:

  • C-<down>: go to next match and open the match.
  • C-<up>: go to previous match and open the match.
  • M-<down>: go to next match without opening the match.
  • M-<up>: go to next match without opening the match.

@tuhdo
Copy link
Contributor

tuhdo commented Apr 22, 2015

@jonathanj the deleting windows issue has just been fixed upstream few minutes ago. Wait a few hours and update Helm again, you won't have that error anymore.

@tuhdo
Copy link
Contributor

tuhdo commented Apr 22, 2015

@jonathanj btw when you want to search for actual key bindings of a major mode, use SPC ?. It will display a list of available key bindings in current major mode and you can play easily narrow to some commands or key bindings you care about by simply typing some pattern. You would find out these key bindings earlier if you had used SPC ?. And press C-z (or Tab in next release) to read the documentation of each command. It's really amazing.

@StreakyCobra
Copy link
Contributor

This is an old issue that seems to have been solved upstream, closing it then, but feel free to open a new issue in case of troubles ☺️

@jonathanj
Copy link
Author

This still doesn't seem to be working for me (I updated all my packages before testing this). The upstream bug originally referenced is still open and there has been no reply from the author to my question.

There also don't appear to be any commits upstream that would have resolved this. Do you mind explaining why you think the issue has been solved? Did I miss something?

@StreakyCobra
Copy link
Contributor

Sorry, I'm doing a cleanup and I may be a little bit to prompt to close 6+ months issues without activity that I can't reproduce. As I failed reproducing it, I thought it was solved. I probably have missed a step.

As you say it's a valid issue, I reopen it ☺️ Sorry for the annoyance.

@StreakyCobra StreakyCobra reopened this Nov 13, 2015
@jonathanj
Copy link
Author

@StreakyCobra I think the trick to reproducing it is in my comment on the upstream issue. You need to restart emacs after flipping that search switch otherwise you're still using the other search mode.

@StreakyCobra
Copy link
Contributor

Indeed, putting (setq evil-search-module 'evil-search) in user-config and restarting allows to reproduce it.

@StreakyCobra
Copy link
Contributor

PR pending #5727

@StreakyCobra StreakyCobra removed their assignment Apr 11, 2016
@d12frosted
Copy link
Collaborator

Fixed with released of Spacemacs v0.200. Let us know if you still have any problems with this issue.

@agzam
Copy link
Contributor

agzam commented Feb 6, 2017

For me it mostly works, I found when it doesn't:

When you do g d evil-goto-definition it highlights it, and then SPC s c does not clear it. (Unless you set: (setq evil-search-module 'isearch)

@d12frosted or someone else, can you guys check it out? It seems evil-commands.el uses isearch for goto-definition

@syl20bnr syl20bnr added Evil and removed Bug :-( labels Feb 7, 2017
@syl20bnr
Copy link
Owner

syl20bnr commented Feb 7, 2017

Thanks agzam, reopening the issue since there are still some bugs.

@syl20bnr syl20bnr reopened this Feb 7, 2017
@duianto
Copy link
Collaborator

duianto commented Sep 5, 2017

Fixed in develop
Clear both / and C-s search highlights with SPC s c #9449

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this issue is still valid!

@github-actions github-actions bot added the stale marked as a stale issue/pr (usually by a bot) label Feb 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Bug tracker - Evil stale marked as a stale issue/pr (usually by a bot)
Projects
None yet
Development

No branches or pull requests

7 participants