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

:set hlsearch not working #481

Closed
jiangmouren opened this issue Jul 20, 2016 · 6 comments
Closed

:set hlsearch not working #481

jiangmouren opened this issue Jul 20, 2016 · 6 comments

Comments

@jiangmouren
Copy link

Do you need help or have a question? Ask us on Slack 👫.
Found a bug? Please fill out the sections below 👍.

What did you do?

I was trying to ":set hlsearch"

What did you expect to happen?

I would my expect my search would be highlighted.

What happened instead?

Not highlighted.

Technical details:

  • VSCode Version:1.3.1
  • VsCodeVim Version: 0.1.3
  • OS: Windows 10
@rebornix
Copy link
Member

I'm working on the vimrc parsing, see #463 and I'll implement set command as well.

But I'm not sure yet if VS Code allows us to do highlighting through API. Will drop more info here later.

@jiangmouren
Copy link
Author

Great! Thanks for the quick respond.

On Jul 19, 2016 8:05 PM, "Peng Lyu" [email protected] wrote:

I'm working on the vimrc parsing, see #463
#463 and I'll implement set
command as well.

But I'm not sure yet if VS Code allows us to do highlighting through API.
Will drop more info here later.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#481 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKj2YaW1Ivl138u45g_3dPEuOW6FmMD4ks5qXZBzgaJpZM4JQU4i
.

@johnfn
Copy link
Member

johnfn commented Jul 20, 2016

Oh, VSCode totally allows you to do that. Not only that, but I actually already show all matches when you are typing with /, I just remove them when you finish.

Should be really easy to add as an option 😄

@rebornix
Copy link
Member

The option will be handled by me.

The question is @jiangmouren can you confirm what @johnfn mentioned, tge highlight should already be there.

@jiangmouren
Copy link
Author

Well the answer is Yes and No.

It is true when you do a search with "/something", the searched patterns
will be highlighted right away(before you hit "enter"). But when you hit
"enter", the highlights go away.

What I really want is like in vim, after you hit "enter", all matched
patterns will be highlighted and then you see where they are in the file
and navigate through them.

Thank you guys!

On Tue, Jul 19, 2016 at 9:01 PM, Peng Lyu [email protected] wrote:

The option will be handled by me.

The question is @jiangmouren https://github.com/jiangmouren can you
confirm what @johnfn https://github.com/johnfn mentioned, tge highlight
should already be there.


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#481 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKj2YdPnq7m_RpNK_iRIi2rlOVVVtf9Dks5qXZ2cgaJpZM4JQU4i
.

@rebornix
Copy link
Member

@jiangmouren just obseved Vim's behavior, it's beyond my expectation. When you enable hlsearch and run search, those matched keys will be hightlighted always untill you disable hlsearch, no matter what mode you are in. Another catch is Vim's search is incremental, this means after you search for something, <esc> wont' quit the search mode. It means if you keep typing chars into the document, newly matched keys will be highlighted and be added to the navigation list.

This is how Vim does and is it a good idea to have it the same way? Actually I'm suprised that you can't even quit a search.

@rebornix rebornix mentioned this issue Jul 25, 2016
10 tasks
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

4 participants