Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add hlsearch #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add hlsearch #1

wants to merge 1 commit into from

Conversation

komainu8
Copy link
Member

No description provided.

@cosmo0920 cosmo0920 requested a review from myokoym December 13, 2017 09:34
@myokoym
Copy link
Contributor

myokoym commented Dec 13, 2017

Thank you for your suggestion!

It may be reasonable, but I feel it's a little hard to clear highlighting by the default way such as :noh.
I think it's better to add a shortcut to remove highlighting.

I have the following plans:

  • A. Add a shortcut to clear highlighting temporarily.
    • e.g.: nnoremap <ESC><ESC> :noh<CR>
  • B. Add a shortcut to toggle enable/disable hlsearch.
    • e.g.: nnoremap <C-h> :set hlsearch!<CR>
  • C. A and B.
  • D. Others.

What do you think about it?

myokoym pushed a commit that referenced this pull request May 11, 2018

Verified

This commit was signed with the committer’s verified signature.
ref: GitHub #1
@myokoym
Copy link
Contributor

myokoym commented May 11, 2018

For now, I've added nnoremap <C-h> :set hlsearch!<CR> because I don't feel like changing the default behavior.

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

Successfully merging this pull request may close these issues.

2 participants