Modify

Opened 10 years ago

Closed 10 years ago

Last modified 9 years ago

#11445 closed enhancement (fixed)

[PATCH] Change POI icons: accommodation and food

Reported by: Yarl Owned by: Yarl
Priority: normal Milestone: 15.08
Component: Internal mappaint style Version:
Keywords: Cc: naoliv

Description

This patch changes icons of accommodation and food group to SVG icons taken from two GitHub repos: gravitystorm/openstreetmap-carto and nebulon42/osmic. Goal is to standardize icons across OSM, if it will be accepted, I will continue with other icons.

Attachments (7)

new-icons-food-and-accommodation.patch (10.9 KB ) - added by Yarl 10 years ago.
patch
accommodation.zip (20.0 KB ) - added by Yarl 10 years ago.
food.zip (14.4 KB ) - added by Yarl 10 years ago.
iconcomp.png (276.9 KB ) - added by Klumbumbus 10 years ago.
carto.png (14.5 KB ) - added by Klumbumbus 10 years ago.
acc.png (3.1 KB ) - added by Klumbumbus 10 years ago.
osmic-josm-shields.zip (108.1 KB ) - added by nebulon42@… 10 years ago.
Osmic icons with white background 2px padding and 3px rounded corners

Download all attachments as: .zip

Change History (42)

by Yarl, 10 years ago

patch

by Yarl, 10 years ago

Attachment: accommodation.zip added

by Yarl, 10 years ago

Attachment: food.zip added

comment:1 by Yarl, 10 years ago

Zip files contains directories from path images/styles/standard. Some files were added, old ones removed.

comment:2 by naoliv, 10 years ago

Cc: naoliv added

comment:3 by Yarl, 10 years ago

Summary: [PATCH] Change POI icons[PATCH] Change POI icons: accommodation and food

comment:4 by Klumbumbus, 10 years ago

Whats the license of these icon? Are they identifiable at a size of 16px?

comment:5 by Yarl, 10 years ago

Both source repos are on CC0.
Icons looks OK for me, exactly like on osm.org.

comment:6 by Klumbumbus, 10 years ago

@team: any general comments to the unification of the icons with the mapnik style?

comment:7 by bastiK, 10 years ago

I like the suggested icons for accommodation, but would prefer the current JOSM icons for cafe, fast food, pub and biergarten.

in reply to:  6 ; comment:8 by bastiK, 10 years ago

Replying to Klumbumbus:

@team: any general comments to the unification of the icons with the mapnik style?

A map is not the same as an editor, so different design goal leads to different icon sets. Colorful icons make sense for JOSM as they are easier to recognize. On a map, you have to care about overall aesthetic look.

in reply to:  8 comment:9 by Klumbumbus, 10 years ago

Replying to bastiK:

A map is not the same as an editor, so different design goal leads to different icon sets. Colorful icons make sense for JOSM as they are easier to recognize. On a map, you have to care about overall aesthetic look.

Sounds reasonable. Some mapnik icons do not work anyway for JOSM like highway=traffic_signals, barrier=gate|lift_gate|bollard, amenity=bank|place_of_worship. I also think it is better to have more colorful icons instead of all shops purple. But I think we can replace some awful JOSM icons (like for optician). Also replacing some png by svg is a good thing.

comment:10 by Yarl, 10 years ago

Maybe I'll add my rationale for this change. Presonally (and I know it's not only my opinion) I prefer unification of icons because it's easier to find in menu and also on map. Frankly, sometimes it's hard to guess what is depicted on current JSOM icons - icons for osm.org are definitely better designed.

I can add some screenshots, but current latest code on repo gives me RuntimeException (Win 8 + NetBeans 8; build log here: http://pastebin.com/Y8LzgVxv)

Edit: most of the icons (all in this change) are SVG so different color for every menu is definitely possible).

Last edited 10 years ago by Yarl (previous) (diff)

by Klumbumbus, 10 years ago

Attachment: iconcomp.png added

comment:11 by Klumbumbus, 10 years ago

comparison screenshoot of current state at http://www.openstreetmap.org/#map=19/50.84661/12.93826


Last edited 10 years ago by Klumbumbus (previous) (diff)

comment:12 by naoliv, 10 years ago

Not so related with this ticket, but the butcher icon is missing in JOSM?

in reply to:  12 comment:13 by Klumbumbus, 10 years ago

Replying to naoliv:

Not so related with this ticket, but the butcher icon is missing in JOSM?

No, it is available.

by Klumbumbus, 10 years ago

Attachment: carto.png added

comment:14 by Klumbumbus, 10 years ago

Another thing is, that you often use aerial imagery background in JOSM and you need colorful icons to see them. Icons with only one color are often hard to see. So we would atleast need a white background for these icons.


comment:15 by Klumbumbus, 10 years ago

Milestone: 15.05

comment:16 by Yarl, 10 years ago

@Klumbumbus: yes, it looks you're right. Testing:

http://i.imgur.com/fYe5Pfa.png

comment:17 by Klumbumbus, 10 years ago

In 8434/josm:

see #11445 - new icon for amenity=restaurant

comment:18 by Klumbumbus, 10 years ago

icon [o31162]

by Klumbumbus, 10 years ago

Attachment: acc.png added

comment:19 by Klumbumbus, 10 years ago

Owner: changed from team to Yarl
Status: newneedinfo

Yarl, can you provide icons for hotel, motel and hostel with white background, where the blue part does not touch the border of the icon? Because with a simple white background attached this is not really an improvement to current state:


in reply to:  19 ; comment:20 by Don-vip, 10 years ago

Replying to Klumbumbus:

Yarl, can you provide icons for hotel, motel and hostel with white background

is it the only remaining thing to do? I'd like to close this ticket in this release :)

in reply to:  20 comment:21 by Klumbumbus, 10 years ago

Replying to Don-vip:

Replying to Klumbumbus:

Yarl, can you provide icons for hotel, motel and hostel with white background

is it the only remaining thing to do? I'd like to close this ticket in this release :)

Yes, but the important part is "where the blue part does not touch the border of the icon"

comment:22 by Yarl, 10 years ago

OK guys, tomorrow there is a bank holiday in Poland, so I'll be offline until end of week. You can wait or close this issue. I'll deliver icons next week.

comment:23 by Don-vip, 10 years ago

Milestone: 15.0515.06

we'll wait, no problem :)

comment:24 by nebulon42@…, 10 years ago

I created Osmic and most of the new icons in osm-carto. To customise the icons from Osmic you can use a script, it is possible to add shields, rounded corners, re-colour, adjust padding, re-size etc. Have a look at https://github.com/nebulon42/osmic/blob/master/tools/export.md. I'm happy to help, just let me know what you need.

comment:25 by Don-vip, 10 years ago

Milestone: 15.0615.07

skip milestone 15.06

by nebulon42@…, 10 years ago

Attachment: osmic-josm-shields.zip added

Osmic icons with white background 2px padding and 3px rounded corners

comment:26 by nebulon42@…, 10 years ago

I have added an archive with all icons that I made with a white background, 2px padding between icon and shield border and 3px rounded corners. This was generated with the following config file: https://github.com/nebulon42/osmic/blob/master/tools/josm-shields.yaml
If you need different parameters or the icons in different form let me know.

comment:27 by Yarl, 10 years ago

Thanks nebulon, here's how they look like on map. I used shield icons for map and icons with transparent backgroud for menu.

http://i.imgur.com/GIa4JNo.png

comment:28 by nebulon42@…, 10 years ago

Looks fine to me. Having all icons in the same (black/grey) colour would also be an option (better contrast on the map).

comment:29 by bastiK, 10 years ago

@Klumbumbus: What is the status?

comment:30 by Klumbumbus, 10 years ago

I will replace some icons when I have some time. Probably tomorrow.

comment:31 by Klumbumbus, 10 years ago

Resolution: fixed
Status: needinfoclosed

In 8592/josm:

fix #11445 - replace some icons, move old icons to josm nodist folder. source of new icons: https://github.com/nebulon42/osmic-josm-style/archive/master.zip, CC0

comment:32 by Klumbumbus, 10 years ago

see also [o31365]

comment:33 by Klumbumbus, 10 years ago

In 8593/josm:

see #11445 - move old icon to nodist folder

comment:34 by Don-vip, 10 years ago

Milestone: 15.0715.08

Milestone renamed

comment:35 by Don-vip, 9 years ago

Component: CoreInternal mappaint style

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Yarl.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.