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

Repeated search of the same text doesn't work #652

Closed
lqlau opened this issue Aug 25, 2016 · 9 comments
Closed

Repeated search of the same text doesn't work #652

lqlau opened this issue Aug 25, 2016 · 9 comments

Comments

@lqlau
Copy link

lqlau commented Aug 25, 2016

What did you do?

Want to search for words in my file.

Put the exact keys you pressed, if relevant.
Pressed "/" to filled in text to search for.
It searched for first matching word.
But when I press "/" again and hit , it should automatically continue searching the file for next match.

What did you expect to happen?

When pressing "/" and hitting it should automatically search for the last text that was searched for.

What happened instead?

Pressing "/" and hitting the second or more time does nothing. You have to manually fill in the same text each time.

Technical details:

  • VSCode Version: 1.4
  • VsCodeVim Version: 0.17
  • OS: Windows 10 64-bit
@xconverge
Copy link
Member

xconverge commented Aug 25, 2016

Reproduced this as well, thanks for reporting! Should be straightforward to remember what was last searched incase an empty search string is used following it.

@johnfn
Copy link
Member

johnfn commented Aug 25, 2016

We haven't even cleared out the search history at this point. This should be like a 3 line change. 😉

@rebornix
Copy link
Member

@johnfn seems we do https://github.com/VSCodeVim/Vim/blob/master/src/actions/actions.ts#L775 .

But it should be still simple as we only need to postpone the reinitialization.

@xconverge
Copy link
Member

I will get to it tonight for sure if it still exists then :)

@johnfn
Copy link
Member

johnfn commented Aug 26, 2016

@rebornix oops, just move that code to here i think https://github.com/VSCodeVim/Vim/blob/master/src/actions/actions.ts#L614

@lqlau
Copy link
Author

lqlau commented Sep 11, 2016

Thanks @xconverge ! Which release will this fix be in?

@xconverge
Copy link
Member

@johnfn can this be reopened or should I file a new issue? This does NOT seem to work now...

@rebornix rebornix reopened this Oct 17, 2016
@rebornix
Copy link
Member

@xconverge let's track it here.

@xconverge
Copy link
Member

I 100% messed this fix up first time around...sorry, no idea what was going on there...

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