Skip to content

Conversation

ecgreb
Copy link
Collaborator

@ecgreb ecgreb commented Mar 24, 2017

Overview

Updates demo app launcher icons and colors

Proposed Changes

screenshot_20170323-222243

screenshot_20170323-222313

screenshot_20170323-222618

screenshot_20170323-222926

@souperneon
Copy link
Member

Everything looks great. I'm sorry, but I am changing my mind about the logo and icon.
Purple on white looks much better! Also, maybe rounded edge square? (Can't believe I just said that!)
Also, a different places logo?

Archive.zip

@sarahsnow1
Copy link
Member

screenshot_20170331-151837

@souperneon I grabbed the new icons/logos from your archive. How do they look to you?

@souperneon
Copy link
Member

These look good! What do you think @ecgreb @sarahlensing?

@ecgreb
Copy link
Collaborator Author

ecgreb commented Apr 3, 2017

Agree it does look good! I just want to double check how the new icon looks on my device which has a white background in the app drawer. Will post images shortly.

Also @souperneon did you want to use rounded corners or stick with them squared?

@souperneon
Copy link
Member

If you can post a screenshot with the rounded? That's the new world ;)

@ecgreb
Copy link
Collaborator Author

ecgreb commented Apr 3, 2017

@souperneon I would be happy to but the zip file attached here still has square corners!

@souperneon
Copy link
Member

Oh, this is not like iOS where the corners automatically round? ;)

@ecgreb
Copy link
Collaborator Author

ecgreb commented Apr 3, 2017

🤖 📵

@ecgreb
Copy link
Collaborator Author

ecgreb commented Apr 12, 2017

@souperneon any update on the rounded mapzen icon?

@sarahsnow1
Copy link
Member

@souperneon how does this look?

screenshot_20170504-151220

@souperneon
Copy link
Member

Beautiful! 👍

@sarahsnow1 sarahsnow1 merged commit 57c8595 into master May 8, 2017
@sarahsnow1 sarahsnow1 deleted the 185-new-logo branch May 8, 2017 15:26
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.

3 participants