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

Replace dialog opens with the wrong field focused #4566

Closed
jcward opened this issue Mar 22, 2016 · 7 comments
Closed

Replace dialog opens with the wrong field focused #4566

jcward opened this issue Mar 22, 2016 · 7 comments
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-find Editor find operations under-discussion Issue is under discussion for relevance, priority, approach
Milestone

Comments

@jcward
Copy link

jcward commented Mar 22, 2016

  • VSCode Version: 0.10.11 (commit: f291f4) downloaded today
  • OS Version: Ubuntu 14.04 x64

Steps to Reproduce:

  1. In the editor, hit CTRL-H for Find & Replace pop-up

Screen grab of the issue:

test

Issue: Notice the Find field should be focused first when the dialog is opened. Instead, the Replace field is focused. And tab moves the focus forward, to the little icons (one would have to know to shift-tab to get back to Find.)

ETA: Ah, I suppose this changed at some point, I was comparing 0.10.11 Linux to 0.10.3 Windows.

So it becomes a question of which implementation is "more better". :p :)

Alright, well, here's a list of editor software that focuses the Find input when pressing Ctrl-H:

  • Microsoft Office
  • Google Docs
  • GitHub in-browser editor
  • Sublime
  • Visual Studio 2015 (community / windows)
  • Atom 1.5.3

Vs. editors that focus Replace first:

  • Atom 1.0.2

Feel free to bring others to my attention.

@Tyriar Tyriar changed the title Linux replace dialog opens with the wrong field focused Replace dialog opens with the wrong field focused Mar 22, 2016
@Tyriar
Copy link
Member

Tyriar commented Mar 22, 2016

This applies across OS's, I also think this is as designed. Might be worth a little discussion with the feature owner though.

Note that you can always use ctrl+f to focus find from anywhere.

@jcward
Copy link
Author

jcward commented Mar 22, 2016

@Tyriar ah, thanks. I've added a list of editor software that focuses on Find first.

@Tyriar
Copy link
Member

Tyriar commented Mar 22, 2016

Atom 1.5.3 focuses find first too.

@OlsonDev
Copy link

OlsonDev commented Jul 5, 2016

I agree, this behavior is counter-intuitive. I realized I run into my issue (of the find widget not trapping focus) a lot because I'm accustomed to hitting Ctrl+H instead of Ctrl+F (because many apps have separate "Find" and "Find & Replace" dialogs). When I Ctrl+H, I notice the focus is on Replace, intuitively hit Tab to go back to Find but then I realize the buttons are tab-focusable (in many IDEs they aren't, just Alt-shortcut-able (Atom & Sublime Text to name a few)). Okay, that's fine. Now, to get back to Find, I have to Shift+Tab twice or Tab a few more times; Shift+Tab is uncomfortable to hit so I'll instead go for 4 more Tabs, just to realize focus has fallen off the find widget.

If this issue was fixed, I probably wouldn't run into my issue as often.

@OlsonDev
Copy link

OlsonDev commented Jul 5, 2016

More examples:

Focuses Find input on Ctrl+H:

  • Beyond Compare
  • WinMerge
  • Notepad++
  • Wordpad
  • Fiddler ScriptEditor
  • SQL Server Management Studio 2016 (Replace widget in query editor)

Focuses Replace input on Ctrl+H:

  • SQL Server Management Studio 2016 (Replace in Files dialog pops up with no query editor open)

@tbfleming
Copy link

It took me time to get used to using ctrl+f to focus the find field and ctrl+h to focus the replace field. Now I prefer it; it beats tabbing.

@alexdima alexdima added the under-discussion Issue is under discussion for relevance, priority, approach label Apr 11, 2017
@alexdima alexdima added this to the Backlog milestone Apr 11, 2017
@rebornix rebornix self-assigned this Apr 28, 2017
@rebornix
Copy link
Member

this issue is talking about the same thing as #17562 . From tomorrow's insiders, if current selection and the existing search string in Find Widget are empty, we'll focus the Find Input first.

Close it first and please let me know if you have any idea about that.

@rebornix rebornix added the *duplicate Issue identified as a duplicate of another issue(s) label May 23, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s) editor-find Editor find operations under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

6 participants