-
Notifications
You must be signed in to change notification settings - Fork 213
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
entrance-alt1 icon #317
Comments
@aparlato We already have an |
@aparlato I noticed that your rounded corners are separate circles: Eventually we will need the icon to be a single path, but the circles are also causing the corners to be a little jagged: And just a minor detail: |
@aparlato @natslaughter none of our icons use perspective (this is in our design guidelines). Do you two feel like this icon can justify the deviation? |
@aparlato |
@aparlato I think it would be nice to use a more contemporary arrow like this, but with a slightly longer stem: Regarding the door, I like the thinner line weight, but the end nubs need to be the same: |
Draft 3 Sorry about that! the missing round curve was an oversight. Let me know what you think of new and improved arrow. Also do you think I should keep the rounded corners on the rectangle considering the new arrow? Or I was thinking of keeping the rounded corners, but getting rid of the rounded line caps. |
@aparlato |
Draft 4 @natslaughter all good, here's a better version of the rounded arrow. |
@aparlato |
@aparlato |
Draft 6 I think this looks pretty good. Two possible concerns:
Thoughts on these concerns? |
@aparlato |
@natslaughter Can do. In the meantime, how do we feel about the 15px more open without the last corners rounding towards each other to better mimic the 11px? |
This was done. |
For id-support from #270
Entrance on taginfo
reference to Maki 1 icon for entrance
reference to current entrance icon
We need an entrance icon for buildings, parking garages, etc. instead of only escalators for subway entrances.
icon for entrance: (key)
cc @nickidlugash
The text was updated successfully, but these errors were encountered: