davesag/sequelize-test-helpers

logo contribution

Closed this issue · 11 comments

reallinfo commented

Hi again @davesag I've reviewed your pinned repositories. i found a logo idea for this repo. What do you think?

sq

davesag commented

Hi @reallinfo that's cool and I like the concept.

Feedback

  1. The perspective seems slightly out to my eye. Maybe drop the top corner slightly.
  2. I think you could choose a better typeface.

Cheers and thanks

D

  • 👍1
reallinfo commented

Yeah, you are right. Thanks for feedback. I made a few changes. How does it look?

sq2

davesag commented

I like the new lower-case font.

To my eye the perspective still looks off. I've hacked it slightly - see below.

logo-tweaks

Not perfect but sorry I am on my phone.

reallinfo commented

Sorry, I don't understand what you're talking about ''slightly?''

  • Are you talking about the corners?
  • colors of the cube?
  • tool color?

Is there the variation you prefer in the following image?

1

davesag commented

The top point is too high. That's what I hacked. I just chopped some of the image off the top edges to fix the perspective. I like the rounded corners but they are not so reminiscent of sequelize. maybe keep the hard corners but just fix the perspective (pull the top point down a bit).

reallinfo commented

Oh okay, i understand now. I made changes.

1

davesag commented

That's cool. Could that top point come up just a tiny bit? Then could I have it as an SVG with the wording all in one line?

reallinfo commented

I made changes. I've uploaded all svg variations here. You can check it here.

https://drive.google.com/open?id=1mY3j3ZXKiWDMeCod4KedHLEMFpvQX1hA

davesag commented

Thanks @reallinfo — I have added the logo. I have also added a thanks to you at the bottom of the readme. Cheers D

reallinfo commented

You're welcome. I will review your other pinned repos. If I get an idea, see you. Thanks!

  • 👍1
davesag commented

You could do a similar one for sequelize-pg-utils