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

Change the default shortcut of Search history navigation to up and down arrows #12645

Closed
sandy081 opened this issue Sep 26, 2016 · 17 comments
Closed
Assignees
Labels
editor-find Editor find operations feature-request Request for new features or functionality on-testplan search Search widget and operation issues
Milestone

Comments

@sandy081
Copy link
Member

Change the default shortcut of Search history navigation to up and down arrows

@sandy081 sandy081 added bug Issue identified by VS Code Team member as probable bug search Search widget and operation issues editor-find Editor find operations labels Sep 26, 2016
@sandy081 sandy081 added this to the September 2016 milestone Sep 26, 2016
@sandy081 sandy081 self-assigned this Sep 26, 2016
@sandy081 sandy081 modified the milestones: October 2016, September 2016 Sep 28, 2016
@sandy081 sandy081 removed the bug Issue identified by VS Code Team member as probable bug label Oct 4, 2016
@sandy081 sandy081 modified the milestones: November 2016, October 2016 Oct 26, 2016
@sandy081 sandy081 modified the milestones: January 2017, November 2016 Dec 7, 2016
@davisein
Copy link

davisein commented Jan 3, 2017

+1 to this change.
In addition, it would be good to have support for cmd+Z or ctrl+Z for undo in the text box.
I thought it was not supported and was looking for it until I found #5508.

@sandy081 sandy081 modified the milestones: February 2017, January 2017 Jan 23, 2017
@sandy081 sandy081 added the feature-request Request for new features or functionality label Feb 2, 2017
@sandy081 sandy081 modified the milestones: Backlog, February 2017 Feb 21, 2017
@petersladek
Copy link

Totally agree. I just had to google to find out Alt+Up/Down works, not intuitive at all. Just copy the experience from sublime text please - including combo selection, autocomplete for folders etc.

@leegee
Copy link

leegee commented May 18, 2017

Almost switched back to Sublime to get this feature, thought I'd file a feature request to have Search History added, got here.

@DanOlivier
Copy link

DanOlivier commented Jul 17, 2017

Uh... Where is the search history for the other fields (files to include and exclude, replacement strings)?
Oh, here it is: #20547.

@campbells
Copy link

+1

Went on the same little goose chase as others to get here. Expected is tabbing for field navigation, up/down for history nav (cf to current line start/end).

@colin-home
Copy link

+1, I didn't realize the search history even existed, since the arrow keys didn't scroll through entries. Even better would be a visible drop-down to pick from, as in feature request #23099.

@ssbarnea
Copy link

ssbarnea commented Jun 1, 2018

I agree on that, I was using this on Atom for years and assumed vscode is missing the feature! Please fix it ASAP and allow us to use UP and DOWN, without having to press any modificators.

@sandy081 sandy081 removed this from the Backlog milestone Jun 1, 2018
@sandy081 sandy081 added this to the June 2018 milestone Jun 1, 2018
@leegee
Copy link

leegee commented Jun 5, 2018

How many votes do we need to get this addressed?

@sandy081
Copy link
Member Author

sandy081 commented Jun 7, 2018

Now you can navigate the history in all input boxes in the following places using Up and Down arrows

  • Search
  • Editor Find
  • Problems
  • Terminal Find

One side affect of this change is that, you can no longer navigate among input boxes in Search view using up and down arrows. But this (navigating) is made consistent with editor find widget.

@ssbarnea
Copy link

ssbarnea commented Jun 7, 2018

@sandy081 This is great. Your remark reminded me about one issue related to navigation between search dialog fields. There are 3-4 buttons that change the search behavior (case, regex, word-match) and those are listed just after the search text. meaning that pressing tab to go to the next field will select them, which is very unlikely what the user may to.

User may want to switch between "search", "files to include" and "files to exclude" using a single TAB and SHIFT-TAB for reverse. I think that the buttons should not be included in the list, or if included they should not be between those 3 major fields.

If needed, I could provide screenshot, just let me know. I think it should be very easy to discover.

@sandy081
Copy link
Member Author

sandy081 commented Jun 7, 2018

@ssbarnea Currently tab and shift tab navigates between search and replace boxes just like editor find widget. I have not included pattern boxes.

@twstddev
Copy link

twstddev commented Jun 8, 2018

@sandy081 I actually have bindings assigned to the nextInputBox and previousInputBox commands. However, I used them to navigate from search input to search results and back :)
Do you think it's still possible to achieve this functionality with the currently exposed commands or with the current api available to extensions?

@sandy081
Copy link
Member Author

sandy081 commented Jun 8, 2018

search.focus.nextInputBox and search.focus.previousInputBox commands exist but without any default keybinding. If you have customised a keybinding for those, they will continue to work.

@ssbarnea You can associate keybindings to above commands to get current navigation with your keybinding.

@roblourens FYI

@twstddev
Copy link

twstddev commented Jun 8, 2018

@sandy081 thank you for the reply.
Unfortunately, I get following error "command 'search.focus.nextInputBox' not found".
Anyway, I was able to get the desired behaviour by remapping my keybinding to list.focusFirst command :)

@sandy081
Copy link
Member Author

sandy081 commented Jun 8, 2018

Changed my mind about defaults. New defaults are cmd+up and cmd+down for navigating among input boxes and results.

@sandy081
Copy link
Member Author

sandy081 commented Jun 8, 2018

@twstddev I did the change to bring them back today. So they will be available in insiders on Monday.

@twstddev
Copy link

twstddev commented Jun 8, 2018

@sandy081 that's awesome. Thanks a lot 👍

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

No branches or pull requests

9 participants