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

status bar git sync icon no spin. #23975

Closed
Moerj opened this issue Apr 6, 2017 · 14 comments
Closed

status bar git sync icon no spin. #23975

Moerj opened this issue Apr 6, 2017 · 14 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug git GIT issues verified Verification succeeded
Milestone

Comments

@Moerj
Copy link

Moerj commented Apr 6, 2017

  • VSCode Version: Code 1.11.0 (9122ec4, 2017-04-05T20:52:01.201Z)
  • OS Version: Darwin x64 16.5.0
  • Extensions:
Extension Author Version
vscode-ipaddress Compulim 0.0.1
beautify HookyQR 1.0.2
code-settings-sync Shan 2.6.2
html-snippets abusaidm 0.1.0
vscode-theme-onedark akamud 1.3.2
project-manager alefragnani 0.13.5
one-monokai-80s axiomaticstudios 1.1.10
theme-material bcmarinacci 1.3.0
path-intellisense christian-kohler 1.2.0
vscode-eslint dbaeumer 1.2.8
githistory donjayamanne 0.2.0
jquerysnippets donjayamanne 0.0.1
theme-dracula dracula-theme 1.8.0
gitlens eamodio 3.2.0
vscode-html-css ecmel 0.0.34
vscode-npm-script eg2 0.1.8
auto-rename-tag formulahendry 0.0.11
ftp-simple humy2833 0.4.16
theme-atomic-dark kimkwanka 0.2.0
HTMLHint mkaufman 0.3.3
vscode-filesize mkxml 1.0.0
theme-oceanicnext naumovs 0.0.4
vetur octref 0.5.6
vuehelper oysun 0.1.0
vscode-icons robertohuertasm 7.5.1
stylelint shinnn 0.23.0
vscode-open-in-github ziyasal 1.2.3

Steps to Reproduce:

1.edit something and click the status bar git sync icon. it no spin.
2.when I use previous version vscode 1.10.2. click it has spin.

@wbercx
Copy link

wbercx commented Apr 6, 2017

Not only does it not spin, it also does not show how many commits you are about to pull in and push up as soon as you click on it. Instead, it instantly changes to show only a non-spinning sync icon.

@joaomoreno joaomoreno added this to the Backlog milestone Apr 6, 2017
@joaomoreno joaomoreno added feature-request Request for new features or functionality git GIT issues labels Apr 6, 2017
@bingzheyuan
Copy link

😭 Sad to find it becomes feature request and moved into backlog. We just want it back!!

@joaomoreno
Copy link
Member

@bingzheyuan Better than just closed. ;)

@pmadhur
Copy link

pmadhur commented Apr 6, 2017

Really sad, it became a feature-request instead of a bug.

@joaomoreno
Copy link
Member

You can upvote this issue: #10455

@elirnm
Copy link

elirnm commented Apr 6, 2017

@joaomoreno Why is this a feature request if the feature already existed in the previous version of vscode? Shouldn't it be considered a regression?

@AliMD
Copy link

AliMD commented Apr 10, 2017

Could you please explain why this must be disabled in new version ?
Thank you

@joaomoreno
Copy link
Member

joaomoreno commented Apr 10, 2017

If you read #10455 you'll discover that it's because extensions are limited to use a fixed palette of icons. The animated icon isn't one of them.

Git is now an extension, by the way.

@wbercx
Copy link

wbercx commented Apr 11, 2017

You say that, but if I am not mistaken - after clicking on the icon - it also used to:

  • Show the amount of commits to be pulled and pushed until the sync had completed. Only then would it be replaced to show just the sync icon.
  • Become less opaque, again until the sync had completed.

Currently though, as soon as you click on the icon, it turns into a state where it looks as if nothing is happening at all. There is more at play here than the fact the icon is not rotating.

Additionally, is the animation not a straight forward CSS rotate? I fail to see what it has to do with icon themes. That issue you referenced is 6 months old. The animation broke last week.

@joaomoreno
Copy link
Member

joaomoreno commented Apr 11, 2017

Git used to be a core component of Code until last month. Core components have the advantage of using a lot of features, including rotating images through DOM access.

As part of the the work that produced an SCM API, which enables many more SCM providers in Code, Git moved to an extension. It now has the same API as any other Code extension. This means it doesn't have access to the DOM. In order to get a rotating icon, we either need to open an incredibly specific rotating icon API, or we think a bit more about this and conclude that #10455 will be a great solution for this problem while enabling others to benefit from it.

Since #10455 is planned for April, I'll just snowplow this one along.

@joaomoreno joaomoreno modified the milestones: April 2017, Backlog Apr 11, 2017
@wbercx
Copy link

wbercx commented Apr 12, 2017

Alright, that makes more sense. Thanks for clarifying!

@joaomoreno
Copy link
Member

Went with the quick n dirty approach. It spins now!

@Moerj
Copy link
Author

Moerj commented Apr 21, 2017

@joaomoreno thank you !

@bpasero bpasero added the verified Verification succeeded label Apr 26, 2017
@AliMD
Copy link

AliMD commented May 6, 2017

cool, updated today and spin back 👍

@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
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

8 participants