Skip to content

added logo and updated nuspecs to point to it#108

Merged
MehdiK merged 2 commits intoTestStack:masterfrom
jason-roberts:NewNugetLogo
Apr 14, 2014
Merged

added logo and updated nuspecs to point to it#108
MehdiK merged 2 commits intoTestStack:masterfrom
jason-roberts:NewNugetLogo

Conversation

@jason-roberts
Copy link
Copy Markdown
Contributor

No description provided.

@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Apr 13, 2014

Thanks @jason-roberts. @mwhelan and I were chatting and we thought since this is a testing suite, it might be better to go with a green color theme to give users a "we're green and you'll be green with us" kinda message!

Thoughts? /cc @robdmoore @JakeGinnivan

@robdmoore
Copy link
Copy Markdown
Member

I do like the purple logo - it looks great. In saying that green might look great too - would love to see what it looks like. The green association with TDD is a nice idea :)

@jason-roberts
Copy link
Copy Markdown
Contributor Author

with a metro green would look like this

untitled

@robdmoore
Copy link
Copy Markdown
Member

I like both so happy with either :)

@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Apr 13, 2014

Honestly I like green better. Purple seems a bit disconnected from a test suite, or maybe I'm overthinking.

Let's see what @JakeGinnivan and @mwhelan think and we can pick the one with more "votes".

@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Apr 13, 2014

Thanks @jason-roberts for creating the green alternative. It's much easier to think when you actually see it :)

@mwhelan
Copy link
Copy Markdown
Member

mwhelan commented Apr 13, 2014

Of the two colours I prefer green. Does this mean a teststack website would
follow a green theme?

On Sun, Apr 13, 2014 at 10:59 AM, Mehdi Khalili notifications@github.comwrote:

Honestly I like green better. Purple seems a bit disconnected from a test
suite, or maybe I'm overthinking.

Let's see what @JakeGinnivan https://github.com/JakeGinnivan and
@mwhelan https://github.com/mwhelan think and we can pick the one with
more "votes".


Reply to this email directly or view it on GitHubhttps://github.com//pull/108#issuecomment-40303827
.

@jason-roberts
Copy link
Copy Markdown
Contributor Author

website wouldn't have to @mwhelan but you could to use the same color for small but important/noteworthy items on the page etc

Green version of logo looks good too - also it's a good color to associate with testing as @MehdiK says

@JakeGinnivan
Copy link
Copy Markdown
Member

I really don't mind, happy to not vote on this :)

@robdmoore
Copy link
Copy Markdown
Member

Cool - green it is?

@MehdiK
Copy link
Copy Markdown
Member

MehdiK commented Apr 14, 2014

I'd vote for it. So if others don't mind it, yes please :)

Thanks guys.

On Mon, Apr 14, 2014 at 12:20 PM, Robert Moore notifications@github.comwrote:

Cool - green it is?

Reply to this email directly or view it on GitHubhttps://github.com//pull/108#issuecomment-40340858
.

@mwhelan
Copy link
Copy Markdown
Member

mwhelan commented Apr 14, 2014

Sounds good!

Sent from my iPad

On 14 Apr 2014, at 08:55, Mehdi Khalili notifications@github.com wrote:

I'd vote for it. So if others don't mind it, yes please :)

Thanks guys.

On Mon, Apr 14, 2014 at 12:20 PM, Robert Moore notifications@github.comwrote:

Cool - green it is?

Reply to this email directly or view it on GitHubhttps://github.com//pull/108#issuecomment-40340858
.


Reply to this email directly or view it on GitHub.

@jason-roberts
Copy link
Copy Markdown
Contributor Author

ok will update to greens

@jason-roberts
Copy link
Copy Markdown
Contributor Author

ok to merge now

MehdiK added a commit that referenced this pull request Apr 14, 2014
added logo and updated nuspecs to point to it
@MehdiK MehdiK merged commit 00a1485 into TestStack:master Apr 14, 2014
@jason-roberts jason-roberts deleted the NewNugetLogo branch April 14, 2014 11:04
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