-
Notifications
You must be signed in to change notification settings - Fork 31k
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
workbench.action.focusRightEditor splits the view when 2 views are open #1891
Comments
This action is the thing that runs on Cmd+3 (Ctrl+3) and is used to bring up a third editor or focus it if 2 are open. This works as designed. |
@bpasero focusThirdEditor is the one that is bound to ctrl+3, it does seem to act exactly the same as focusRightEditor though. Given that ctrl+3 already acts like this, this keybinding doesn't seem to provide much utility at all? A user could use ctrl+1/2/3 to focus all 3 editors or ctrl+shift+alt+left or right to only focus the left most or right most? |
@Tyriar sorry I confused the actions, "Focus into Next Editor on the Right" is distinct from Cmd+3. For me it works like this:
What puzzles me is why no editor opens for you when 1 is open? Btw in your example the second editor already has focus when you call the split action. That is why a third editor opens. If you give focus to the first editor and then invoke the action, it works as expected. |
@bpasero It appears I got a little confused too, I just looked again and I think I was indeed running
|
Yes, that explains it. Fuzzy find is not enabled in anything but the file picker (opt in). |
@bpasero it did do a fuzzy match though on "Focus into Right Hand Editor" |
That is actually a conicidence from our camel case matcher :) |
Ah, well no issue apart from #509 then :) thanks for looking into it. |
👍 |
Steps to reproduce
"focus right"
Expected
The second text editor on the right is focused.
Actual
A third text editor is split (from the second text editor) and focused.
Notes
workbench.action.focusRightEditor
does nothing when there is only a single editor.workbench.action.focusRightEditor
appears to function correctly when there are 3 text editors, just not 2.The text was updated successfully, but these errors were encountered: