-
Notifications
You must be signed in to change notification settings - Fork 824
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
add amenity=charging_station icon #2239
Conversation
The icon fits in and the PR looks technically okay, but we never decided in #989 if we want to render charging stations or not. Having the PR lets us see how it looks. |
I don't have an opinion if we should render it or not, but the internal part of the icon (spark) is not clear to me on all these exports - could it be wider a bit? |
I like wider more, but I think spark could be longer on top (not truncated like it is now). |
With wider and taller version the sparks are perfectly recognizable for me now, but I think they could be placed a bit lower. |
With that version the bolt is two pixels from the top and three from the bottom. It would look weird below center. |
I don't think it would, but don't care that much for this detail - it's already an improvement to me. |
sent from a phone
I think wider is better recognizable, maybe you could even try to get rid of the pump and just keep the jack and the bolt (bolt in blue and slightly bigger in this case) |
Another way could be to get rid of the bolt and give more focus to the plug? |
I agree, the wider/taller bolt is easier to see. The icon we have for fuel has a pump and gas hose, so it'd be nice if the electric charging icon was "similar but different". Looking around the internet I'm seeing similar lightning bolt on a "pump" with cord and plug design that was used in this icon. So it seems to be fairly standard and it would be good to stick with that. It'd be nice to avoid the "bike shed effect", but if any of you want to explore other ideas feel free to mock them up so we can see. |
Sorry, it's my last post. The icon is good and I don't know if it could be better but just to explain. |
I would propose an icon with a bigger plug, which is already in use in this POI map: http://www.flosm.de/html/POI-Karte.html#269265.97/49.4232406/6.96490407/0/chargingcar/chargingstation and also used by Osmand: https://github.com/osmandapp/OsmAnd-resources/blob/master/icons/svg/transport/charging_station.svg |
should be added and the bolt symbol is good enough at the moment and should not stop it |
Are you sure the icon is pixel-aligned? It looks a bit blurry to me. |
Merged, thanks @davidfg4. Problems with the icon, if any, could be fixed in a follow-up PR. |
Thanks @math1985. It is aligned to pixels, but in order to differentiate between the cable the the prongs, the cable is a bit thicker than 1px, and the prongs are slightly less. Again, I didn't design this, it's from https://github.com/gmgeo/osmic |
Am I right that merged version is the same as original (not wider and taller)? |
@kocio-pl yes |
I thought this:
meant you wanted to replace the original with it. Do you really want to do that (and haven't done it yet) or I misunderstood you and you want to stick with it? |
I'm perfectly happy with the original icon too, so at this point I'm not going to make a new PR to change it. |
I'd like to make some mockups and PR then. Could you share the link to the SVG version? I'm against bikesheding and also like to use osmic if possible, but I prefer readability over pureness. |
Sure, here is the larger bolt: http://gianforte.org/charging_station_LARGER_BOLT.16.svg |
Addresses #989 and #108
http://wiki.openstreetmap.org/wiki/Tag:amenity%3Dcharging_station
This adds an icon for amenity=charging_station. If there is a name it should also be rendered, although very few charging stations have a name tag. The icon was taken from https://github.com/gmgeo/osmic per @nebulon42 's suggestion.
There are currently over 8000 charging_station points, and it would be good to be able to view them on the map. See http://overpass-turbo.eu/s/hr4 for where these are currently.
17:
18:
19: