Skip to content

Conversation

@KaiSforza
Copy link
Contributor

Fixes #2280

pull-request.png was extremely similar to the standard "Source Control" tab in vscode, this changes it to one that is much more github.

Dark theme:
image

Light theme:
image

Light sidebar:
image

@KaiSforza
Copy link
Contributor Author

This seems to have been the icon at one point when looking at the readme's first gif

@KaiSforza
Copy link
Contributor Author

Any comments on this?

@RMacfarlane
Copy link
Contributor

Hey @KaiSforza, sorry for the delayed response!

I agree that this icon does a better job of representing the extension and is also easier to distinguish from the other icons in the activity bar. The reason this icon was changed is because of GitHub codespaces, which introduces its own GitHub icon on the activity bar:

Screen Shot 2020-11-30 at 8 15 30 AM

Maybe an icon that combines the GitHub logo and the git pull icon would work for this scenario, like having the GitHub logo with it at the corner as the codespaces icon does with the VSCode logo. @misolori Do you think that would work?

@miguelsolorio
Copy link

@RMacfarlane I think we'd need to ask the GitHub team as that is modifying their brand logo. My thought is they wouldn't be keen on it since it's not a product/feature.

@dantman
Copy link

dantman commented Feb 2, 2021

The current icon is actively harmful. It's very hard to distinguish the sidebar panel for making commits (which you need to use very frequently) from the GitHub issues/PR sidebar.

Copy link
Contributor

@RMacfarlane RMacfarlane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @KaiSforza, going to go ahead and merge this now since Codespaces no longer has a conflicting icon!

@RMacfarlane RMacfarlane merged commit 497a631 into microsoft:master Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace Icon on the Activity Bar by an Octocat (or anything different to built-in Source Control)

4 participants