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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add new header to readme #708

Merged
merged 7 commits into from
Jun 19, 2019
Merged

Add new header to readme #708

merged 7 commits into from
Jun 19, 2019

Conversation

dnywh
Copy link
Contributor

@dnywh dnywh commented Jun 17, 2019

馃摬 What

Replace the old green wordmark with the new green app icon (#707).

馃 Why

  • Old green is going away
  • Why not show off the app icon a lil

馃洜 How

Changing the SVG to a PNG (because of shadows).

馃憖 See

Before 馃悰 After 馃
before after

馃摑 Notes

Added @justinswart as a reviewer as he added the logo originally and has been dealing with my app icon madness.

Replaced the old green wordmark with the new green app icon from #707
@dnywh dnywh requested a review from justinswart June 17, 2019 19:38
@justinswart
Copy link
Contributor

Hey @dannyalright I'm actually in favour of keeping the wordmark and maybe just changing the colour? Unless you have strong feelings about replacing it with the icon?

@dnywh
Copy link
Contributor Author

dnywh commented Jun 18, 2019

Copy that.

No strong feelings鈥擨 was going for more of the app vibe over the company vibe. I'll update the colour of the wordmark, maybe shrink it a bit, and then update the PR.

@dnywh
Copy link
Contributor Author

dnywh commented Jun 18, 2019

how's that, @justinswart ?

@justinswart
Copy link
Contributor

Yeah I like that - do you want to open one on https://github.com/kickstarter/android-oss too, that repo is currently the same as this one?

@eoji you dig?

@eoji
Copy link
Contributor

eoji commented Jun 18, 2019

yeah @dannyalright and I are pairing on the new icon tomorrow AM!

@dnywh dnywh changed the title Add new app icon to readme Add new header to readme Jun 19, 2019
@dnywh dnywh merged commit 25e2883 into master Jun 19, 2019
@dnywh dnywh deleted the app-icon-readme branch June 19, 2019 18:55
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

3 participants