Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Use OpenWebSite image for View/Open on GitHub #1877

Merged
merged 3 commits into from
Aug 29, 2018

Conversation

jcansdale
Copy link
Collaborator

@jcansdale jcansdale commented Aug 23, 2018

Breaking up commits from #1561 into separate PRs...

What this PR does

The GitHub logo is currently overloaded for different commands. This PR changes the View/Open on GitHub to use the standard OpenWebSite image.

  • Use OpenWebSite image for View/Open on GitHub

Before
image

After
image

Copy link
Contributor

@donokuda donokuda left a comment

Choose a reason for hiding this comment

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

🙇✨

The GitHub logo is currently overloaded for different commands. This PR changes the View/Open on GitHub to use the standard OpenWebSite image.

Definitely agree on the GitHub mark being overloaded with different commands. 👍 We might want to decide on a single command for the GitHub mark, or try to avoid using it as a command icon in general.

Thanks Jamie!

@jcansdale jcansdale requested review from tinaschrepfer and removed request for tinaschrepfer August 28, 2018 17:24
@jcansdale jcansdale merged commit c4474f0 into master Aug 29, 2018
@jcansdale jcansdale deleted the jcansdale/OpenWebSite-image-ViewOnGitHub branch August 29, 2018 17:53
@meaghanlewis meaghanlewis added this to the 2.5.6 milestone Aug 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants