Vscode: SCM: Support keyboard shortcuts

[kgrossjo]

• VSCode Version: 1.11.2
• OS Version: macOS Sierra 10.12.4

Steps to Reproduce:

1. Open a project under git, with some changed files
2. Ctrl-Shift-G to open the SCM view. It has a commit message at the top and a list of Changes below.
3. Tab to go to the list of Changes.
4. Cursor down until an interesting file is selected.

Now I can do Return to see a diff. After that, the file is highlighted and there are icons in the SCM view for reverting this file (curved arrow) or for staging it (plus sign).

But I can't find a keyboard shortcut for the "stage" icon. I tried Return, Ctrl-Return, Space, Ctrl-Space, S, Shift-S, the Plus key, Cmd-Down (because that opens a file in Explorer view), Cmd-Up, Cmd-Right, Cmd-Left.

I also tried Cmd-Shift-P to find the command, but it doesn't have a keyboard shortcut printed. I also tried to right-click the file, which gave me a context menu (with "stage" in it), again without keyboard shortcut.

[joaomoreno]

Try Shift F10 and then pick the desired action.

[kgrossjo]

Hm. F10 invokes a macOS action (to show all windows of the application), and Shift-F10 does the same but with slow motion animation.

But it provided a starting point: In the keyboard shortcuts I searched for F10, found the Shift-F10 command, and reassigned it to cmd-F10. So now when I'm in a text editor, I can use it and I get a nice context menu.

But the keyboard shortcut has "When editorTextFocus", and so it doesn't work in the SCM view.

I also searched the reference documentation to see if there is a "When" setting (I think it's called "context"?) that I can use to trigger when the SCM view has focus. But I've not been able to find it.

The issue was closed, but for me, it's not resolved yet. @joaomoreno is it okay with you to reopen this ticket?

[joaomoreno]

Absolutely.

Yeah that keybinding is only valid for the editor, not the list widget. You should use Fn-F10, though. Does that work for you?

[kgrossjo]

Fn-F10 mutes the loudspeakers. Fn-Shift-F10 seems to do the same.

By now, it's starting to look as if these actions aren't possible via the keyboard. There is an action for "git add" that I can activate via cmd-shift-P, but that one seems to apply to the currently focused editor, not to the file that's selected in the SCM view.

Thanks for your help, @joaomoreno I really appreciate it. I wish I could be more positive in my messages.

[joaomoreno]

No worries @kgrossjo. We should definitely get this story better.

[cleidigh]

@kgrossjo
@joaomoreno
I think my PR #24860 addresses this albeit without a specific shortcut.
it allows The Tab Key to be used to get to the ad and stage icons on the focused line/file
without having to press Enter to select the line which would open the diff in the
editor.

[yetithefoot]

I came from Sublime Text. I'm using GitSavvy plugin for ST, which implements great user experience working with git via shortcuts. It would be possible to implement similar experience via VS Code keybindings, but I can find specific actions and options to specify when scope for scm viewlet. Appreciate any help.

https://cloud.githubusercontent.com/assets/5016978/6704171/2f236466-cd02-11e4-9b7d-22cc880b5e9d.png

[franciscolourenco]

Gitsavvy is by far the best git staging experience I've tried so far. The only thing I'm missing in VSCode at the moment.

Some of the features missing:

• [ ] Open file diff view from directly from editor
• [ ] Ability to stage entire hunks, without having to select them
• [ ] Good Shortcuts, ideally without having to press modifier keys

[AB1908]

I similarly came from Atom where you can press Enter on the focused file to Stage/Unstage it. It would be great to have a keyboard shortcut for this. Perhaps we could create a keybinding since the command to stage a file is already in the command palette?

[LFDM]

I used an own fork of VSCode for the last to years, where I heavily modified this panel to work in the following way (worth noting that this was also all heavy inspired by vim keybindings):

• When you open the SCM view, the input field is NOT focused, we instead are directly focused at the first unstaged file (I use alt+g to open the SCM view.
• j and k allows to navigate through the list of files
• a adds the file to the staged changes, if its already staged it unstages
• u undos changes to a file
• d diffs the file
• i jumps to the input field to write the commit message
• esc jumps out of the input field back to the file list.
• ctrl/cmd+enter commits it.

j would also select the group headers like "Changes" - pressing a there would stage all changed files, or unstage all files when you have "Staged Changes" focused.

So the regular flow is for example like this:

alt+g to open. j j j to jump to the file I want, a to stage the file, a to stage the next file, a to stage the next file, i to start writing the commit message, ctrl+enter to commit.

It's blazingly quick and super convenient. Definitely something I am missing from regular VSCode.

[kgrossjo]

It turns out that quite a few things have changed, usability is much improved. It's still possible to do better, though.

Say I'm in a project that has a git repo and some files are changed ("unstaged changes" in git status).

ctrl-shift-g opens the SCM view, the commit message field is focused. I can do tab to go to the list of files ("Changes"), then j to the file I want, then tab tab tab to focus the "+" icon, then space to add the file to the list of "Staged Changes".

But the SCM view has lost focus, so I have to ctrl-shift-g again, which focuses the commit message field. From there, I can tab to the list of files, then j my way down over all the files I've already staged down to the unstaged changes, to the file I want.

I feel that it would be nice if, after hitting space to execute the "+" action to move the file to the list of staged changes, the SCM view retained focus, and perhaps the first unstaged file could have focus, or one file down from the one I "+"-ed, or one file up from the one I "+"-ed.

I also like the approach presented by @LFDM in his comment from a few hours ago. That after ctrl-shift-g, the list of files has focus, and that there is a key to move focus to the commit message.

[simkimsia]

I tried this but didn't work as well.

WOuld be good if there's a clear way on how to set the when expression for scm view is in focus

[electrovir]

Is this dead? :slightly_frowning_face: Reviewing commits that touch many files in VS Code is a very slow experience right now...

[CombeeMike]

Since my feature request #50434 has been closed as a duplicate of #24389 which then has been merged with this one, I'd like to stress here that a dedicated command (with shortcut) to open the next file with changes in the currently open diff view would improve my VCS/GIT workflow tremendously (as described in #50434)!

Some additional thoughts:

• If no diff view is open, show the Source Control panel & open a new diff editor with the first file with changes
• If there are no more files with changes (the last one in the list is reached), either show the first one again or show some according message/toast to the user. I'd prefer the second option since it makes it a bit clearer that I've reviewed all my changes.

Workaround using AHK

In the meantime I managed to workaround this by using some AHK script which is bound to a global Windows hotkey:

; ======================================================================================================================
; Select the next or previous item relative to the current selection in the "changed items" list and open the diff
; editor
; ======================================================================================================================
SelectNextOrPrev(SelNext)
{
  Send ^+m   ; Ctrl+Shift+m -> Open "Source Control" panel
  Send {Tab} ; Go to list of changed files

  ; Select next or prev file in list
  If (SelNext == True)
    Send {Down}
  Else
    Send {Up}

  Send {Enter} ; Show diff editor
}

; ======================================================================================================================
; Select the first item in the "changed items" list and open the diff editor
; ======================================================================================================================
SelectFirst()
{
  Send ^+m          ; Ctrl+Shift+m -> Open "Source Control" panel
  Send {Esc}        ; Remove focus from input box
  Send {Home}       ; Set active item to the parent of the input box
  Send {Down}{Down} ; Select first item in list
  Send {Enter}      ; Show diff editor
}

; ======================================================================================================================
; Main
; ======================================================================================================================
If WinActive("ahk_exe Code.exe")
{
  arg = %1% ; Accepted values for command line arg0: "f" = Show first, "n" = Show next, "p" = Show prev
  If (arg = "f")
    SelectFirst()
  Else
    SelectNextOrPrev(arg = "n")
}
Else
{
  MsgBox, Not in VSCode!
}

This works pretty well for me so far but I'm sure that there are situations where this doesn't work as expected. E.g. I know that the script to select the first item in the list doesn't work, when the "changes" list is collapsed etc.

[johndevedu]

This has been bugging me for a while. Finally came up w/ a solution that works pretty well. TL:DR instead of navigating through each file one by one in the SCM panel, get all files open (requires GitLens extension) then review one file at a time -> close tab when done.

1. Close all tabs (optional, but cleaner experience for me)
2. Use GitLens: Open All Changes (with difftool). I have difftool connected to VSCode.
   {
   "key": "alt+numpad6",
   "command": "gitlens.externalDiffAll"
   }
3. Show Next Change / Show Previous Change
   {
   "key": "alt+numpad3",
   "command": "workbench.action.compareEditor.nextChange",
   "when": "textCompareEditorVisible"
   }
4. Once done reviewing each file, close tab
5. Repeat steps 1-4 for other files until all files complete.