Skip to content

Change icon to more representative icon - #2294

Merged
Rachel Macfarlane (RMacfarlane) merged 1 commit into
microsoft:masterfrom
KaiSforza:githubicon
Feb 11, 2021
Merged

Change icon to more representative icon#2294
Rachel Macfarlane (RMacfarlane) merged 1 commit into
microsoft:masterfrom
KaiSforza:githubicon

Conversation

@KaiSforza

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@KaiSforza

Copy link
Copy Markdown
Contributor Author

Any comments on this?

@RMacfarlane

Copy link
Copy Markdown
Contributor

Hey Kai Sforza (@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
Copy Markdown

Rachel Macfarlane (@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
Copy Markdown

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
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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

@RMacfarlane
Rachel Macfarlane (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