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

Changed Rust Icon #520

Closed
wants to merge 1 commit into from
Closed

Changed Rust Icon #520

wants to merge 1 commit into from

Conversation

CDMcGwire
Copy link

The rust icon seemed kind of noisy, so I took an SVG of Ferris the Rustacean and modified it for better appearance at a small size. I tried to make sure it fits within the style of the package as a whole while being distinctly 'Rust.'

Let me know if I need to do anything more to integrate it properly. I figured modifying the existing SVG file and changing the color in the mapping.less would be sufficient. I don't actually use atom, I'm here from Visual Studio Code, which apparently adopted them as well. 🤷‍♂️

Either way, like the project!

The rust icon seemed kind of noisy, so I took an SVG of Ferris the Rustacean and modified it for better appearance at a small size. I tried to make sure it fits within the style of the package as a whole while being distinctly 'Rust.'

Let me know if I need to do anything more to integrate it properly. I figured modifying the existing SVG file and changing the color in the mapping.less would be sufficient.
@orta
Copy link
Collaborator

orta commented Jul 8, 2019

Welcome to a blast from the past on this PR - so, I don't think this SVG renders how you want:

Screen Shot 2019-07-08 at 8 11 16 AM

It's kinda like a sad crab 😄.

I am open to this though! I'll do a quick twitter poll on what people think too.

What it looks like now:

Screen Shot 2019-07-08 at 8 05 16 AM

vs

Screen Shot 2019-07-08 at 8 17 19 AM

@orta
Copy link
Collaborator

orta commented Jul 18, 2019

Alright, looks like with an almost 50% majority, keeping it the same has won

@orta orta closed this Jul 18, 2019
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.

None yet

2 participants