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

change th to cubes for icon to more accurately reflect text #364

Merged
merged 1 commit into from
Aug 26, 2016

Conversation

abemassry
Copy link
Contributor

Before:
screen shot 2016-08-23 at 4 20 25 pm

After:
screen shot 2016-08-23 at 4 20 32 pm

@ddevault
Copy link
Contributor

Can we get this to use a Linode L instead?

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.759% when pulling d32c057 on abemassry:linodes-to-cube into 910b8b5 on linode:master.

@eatonphil
Copy link
Contributor

The cubes are cuter.

@ddevault
Copy link
Contributor

That's not the Linode L :/

@eatonphil
Copy link
Contributor

I'm not a graphic designer :/

@eatonphil
Copy link
Contributor

+1 for the cubes

@ddevault
Copy link
Contributor

I meant literally using a greyscale version of our logo. There's an SVG around somewhere

@eatonphil
Copy link
Contributor

If someone stumbles on it, sure. In the meantime, this cleans up that icon so we can reuse it in login instead.

@ddevault
Copy link
Contributor

Stumbled upon

@eatonphil
Copy link
Contributor

Can you rip off the linode title and replace the green with white?

@ddevault
Copy link
Contributor

Tomorrow, sure.

@eatonphil
Copy link
Contributor

nice

@na3d
Copy link
Contributor

na3d commented Aug 24, 2016

Yeah, I am sure Rob has an EPS file we can use.

@ddevault
Copy link
Contributor

@ddevault ddevault merged commit 2024131 into linode:master Aug 26, 2016
@abemassry abemassry deleted the linodes-to-cube branch December 2, 2016 20:37
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.

5 participants