microsoft / vscode

Visual Studio Code
https://code.visualstudio.com
MIT License
160.7k stars 28.15k forks source link

Synchronize change icon does not move when pulling from Command pane #29944

Closed fbnlsr closed 5 years ago

fbnlsr commented 7 years ago
Extension Author Version
EditorConfig EditorConfig 0.9.3
crane HvyIndustries 0.3.6
vscode-cdnjs JakeWilson 0.14.3
autofilename JerryHong 0.1.3
html-snippets abusaidm 0.1.0
Bookmarks alefragnani 0.15.2
vscode-twig-pack bajdzis 1.0.0
toml be5invis 0.0.3
vscode-dash deerawan 1.6.0
theme-dracula dracula-theme 2.1.3
php-intellisense felixfbecker 1.4.3
vscode-npm fknop 3.3.0
font-awesome-codes-css medzhidov 0.1.1
font-awesome-codes-html medzhidov 0.0.7
seti-icons qinjia 0.1.3
vscode-gulp tanato 0.0.4
gitblame waderyan 1.11.3
vscode-todo-highlight wayou 0.5.5 ;

Steps to Reproduce:

  1. Use "Git pull" in Command pane (Cmd + Shift + P)
  2. Sync icon does not turn

Simple issue, more cosmetics than anything, but when using commands such as Git pull or Git push the sync icon in the status bar should move the same way than when it's clicked on.

hamxabaig commented 7 years ago

I'd like to work on it. But i'm confused on how we can check if the command finished when someone typed in the integrated terminal ? Do we have something for that ?

fbnlsr commented 7 years ago

@hamxabaig thanks a lot for looking into this issue. I'm not sure if there's a way to get a feedback from the integrated terminal when typing git commands.

If you do a git pull from the command pane, there's a visual feedback on the SCM icon in vscode's sidebar. However, if you use git pull from the integrated terminal, that visual feedback is not present.

I think we should consider the integrated terminal as a completely separate entity, especially considering it provides its own feedback.

The problem my issue raises is the lack of visual feedback consistency between clicking the sync icon in the status bar and using git pull in the command pane.

hamxabaig commented 7 years ago

@fbnlsr I got it what you've in your mind. But what i'm saying is there any event fired when git pull is run ? That i can use to find out if user typed git pull in the command line. Any help would be great. Thanks

joaomoreno commented 7 years ago

@hamxabaig Ignore the integrated terminal. Check this out.

hamxabaig commented 7 years ago

@joaomoreno Thanks for pointing me, I'd have a look into this today when i get time. 👌

joaomoreno commented 5 years ago

This is fixed.