Skip to content

Add logo #13

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

Closed
wants to merge 2 commits into from
Closed

Add logo #13

wants to merge 2 commits into from

Conversation

Toinane
Copy link
Contributor

@Toinane Toinane commented Oct 19, 2017

#13 - l've made some logo :) :

  • one in colors of the app
  • the same in white version

@itaditya
Copy link
Owner

Wonderful logo, however I need something a little different. Can you please read this #10 (comment) .

In total try to add text in the logo also preferrably alongside it.

@Toinane Toinane closed this Oct 21, 2017
@Toinane Toinane reopened this Oct 21, 2017
@itaditya
Copy link
Owner

Hey I didn't find anything new.. Have you took my points into consideration??
Also place the logo in client folder

@Toinane
Copy link
Contributor Author

Toinane commented Oct 22, 2017

I work too, I can't work on open-source everyday.
I make another version, but I can't spend much time on design

@itaditya
Copy link
Owner

I thought, you had opened a new PR, but then I checked and there were no commits. That's why I asked. Totally my mistake, really sorry.
Also keep up the good work

@Toinane
Copy link
Contributor Author

Toinane commented Oct 22, 2017

no worries :) can you explain more what you want to change or what I keep?

@itaditya
Copy link
Owner

itaditya commented Oct 22, 2017

Attach a screenshot hinting the possible location for this awesome logo, you probably would have something in mind while designing the logo

@Toinane
Copy link
Contributor Author

Toinane commented Oct 25, 2017

I haven't no times to design actually :/ and my only way to design is at my work, so i will close this PR but you can use my previous work as you want!

@Toinane Toinane closed this Oct 25, 2017
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.

2 participants