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

There is no search history #5508

Closed
brunolemos opened this issue Apr 19, 2016 · 12 comments
Closed

There is no search history #5508

brunolemos opened this issue Apr 19, 2016 · 12 comments
Assignees
Labels
feature-request Request for new features or functionality search Search widget and operation issues

Comments

@brunolemos
Copy link

Can't easily redo previous search, which is very annoying and decreases productivity.

  • VSCode Version: 1.0.0
  • OS Version: OS X 10.11.4

Steps to Reproduce:

  1. Press Shift+Cmd+F
  2. Search for something
  3. Search for another thing
  4. Press UP key (or Cmd+Z or whatever)

Expected: Show previous search term, like all other editors
Result: Nothing happens

VS Code:
VS Code Search Example

Atom:
Atom Search Example

@dbaeumer dbaeumer added the search Search widget and operation issues label Apr 19, 2016
@dbaeumer dbaeumer added the feature-request Request for new features or functionality label Apr 19, 2016
@dbaeumer
Copy link
Member

@kieferrm heard you are taking over search :-)

@JimmyBurdette
Copy link

I really liked this feature in Atom and would use it regularly.

@Urik
Copy link

Urik commented Sep 11, 2016

I would love to give this feature a shot. Any chance you'd accept a contribution on this one?

@sandy081 sandy081 modified the milestones: September 2016, Backlog Sep 12, 2016
@sandy081
Copy link
Member

sandy081 commented Sep 12, 2016

@Urik We are open to contributions, but sorry to say that I have already started on this and planning to push it today. If you are interested I can point you to other interesting features. Thanks.

@Urik
Copy link

Urik commented Sep 12, 2016

That's so great to hear Sandy, I've been lusting over that feature since I started working with VSCode!
I'd love to see these possible features, thanks.

@sandy081
Copy link
Member

Implemented following support:

Search input in Search view
Find input in Find widget

  • alt+up : Shows the previous term in history
  • alt+down : Shows the next term in history

Customize key board short cuts

  • search.history.showPrevious : alt+up
  • search.history.showNext : alt+down
  • find.history.showPrevious : alt+up
  • find.history.showNext : alt+down

@hipstersmoothie
Copy link

this work well but i feel its kinda hidden behind the alt+. is there any way to have it just be up/down on the search fields?

@sandy081
Copy link
Member

@hipstersmoothie You can customize the key board short cuts overriding the commands and keys mentioned in comment

@gandhis1
Copy link

Thats great, still, the defaults should be intuitive and natural. Not sure that the use of ALT satisfies that. Is there any downside to not using ALT? These are single line queries (absent the use of \n), anyway, so I would think up and down do not have a meaning in these contexts.

@chrmarti chrmarti mentioned this issue Sep 26, 2016
1 task
@sandy081
Copy link
Member

@gandhis1 Up and Down are already used to navigate between the input boxes (Search, Replace, Files include, Files exclude). Using Alt goes with other key board short cuts for actions - Toggle match case, reg ex, whole word.

I am open for the suggestions to change the default one to more better ones.

Thanks for the feedback

@dmythro
Copy link

dmythro commented Sep 26, 2016

@sandy081 navigation between fields is actually expected by tab, shift+tab. So I don't see any reason to not just use arrows for search/replace history in those fields.

@sandy081
Copy link
Member

Thanks for the feedback. Filed the following issue to change it

#12645

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality search Search widget and operation issues
Projects
None yet
Development

No branches or pull requests

9 participants