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

'Git: Open Changes' doesn't focus the newly opened tab with the diffs #32816

Closed
vvs opened this issue Aug 19, 2017 · 5 comments
Closed

'Git: Open Changes' doesn't focus the newly opened tab with the diffs #32816

vvs opened this issue Aug 19, 2017 · 5 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug git GIT issues verified Verification succeeded
Milestone

Comments

@vvs
Copy link

vvs commented Aug 19, 2017

  • VSCode Version: Code - Insiders 1.16.0-insider (aebaece, 2017-08-18T05:22:55.860Z)
  • OS Version: Windows_NT x64 10.0.15063
  • Extensions:
Extension Author (truncated) Version
Bookmarks ale 0.15.2
project-manager ale 0.19.0
vscode-eslint dba 1.2.11
gitlens eam 4.4.0
EditorConfig Edi 0.9.4
prettier-vscode esb 0.22.1
auto-close-tag for 0.4.3
beautify Hoo 1.1.1
atom-keybindings ms- 3.0.1
vetur oct 0.9.4
stylelint shi 0.28.0

Steps to Reproduce:

  1. Open the command pallet.
  2. Invoke "Git: Open Changes" (there must be some outstanding changes in the current git repo)
  3. The new tab with the changes opens.
  4. The tab is NOT focused, so I had to grab the mouse and perform the click. Without focus, the keyboard navigation in the diff view is not working (like the up/down arrow, for example).
  5. I have some extra keybindings, specially for this diff view, and they don't work as well, until I focus the diff view.

This seems to be a regression, it worked fine earlier.

Reproduces without extensions: Yes

@vscodebot vscodebot bot added the insiders label Aug 19, 2017
@vscodebot vscodebot bot added the git GIT issues label Aug 19, 2017
@joaomoreno
Copy link
Member

This came with #27411

We should be able to know when this happens and set focus accordingly.

@joaomoreno joaomoreno added this to the August 2017 milestone Aug 21, 2017
@joaomoreno joaomoreno added the bug Issue identified by VS Code Team member as probable bug label Aug 21, 2017
@vvs
Copy link
Author

vvs commented Aug 23, 2017

@joaomoreno I can still reproduce (a slightly-different) the problem on both, the latest VS Code Insiders and the freshly built from source as well.

Please note that I am using the keyboard shortcut for the "git.openChange". In my case it is alt+d.

So, currently, when I press alt+d on a file that has changes in it, a new tab with diffs opens, but not focused.

Should I file a separate bug for that?

I did notice that when using the command palette to "Git: Open Changes", focus works, so this part is fixed.

@joaomoreno
Copy link
Member

I actually pushed a change today which should address that too. Let me know with tomorrow's update if that fixes it for you.

@vvs
Copy link
Author

vvs commented Aug 23, 2017

Thank you, it works for me when VS Code is built from the sources! So I assume it will continue to work in tomorrow insiders build as well :)

👍

@joaomoreno
Copy link
Member

Sweet, great to know. Thanks!

@bpasero bpasero added the verified Verification succeeded label Aug 31, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 17, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug git GIT issues verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants