Modify

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13217 closed enhancement (fixed)

replace png icons by svg from osm svn source, sort icons and remove duplicates

Reported by: Klumbumbus Owned by: Klumbumbus
Priority: normal Milestone: 16.07
Component: Core Version:
Keywords: Cc:

Description (last modified by Klumbumbus)

Attachments (1)

old_and_new_icon_paths.xlsx (27.4 KB ) - added by Klumbumbus 8 years ago.
404 new icon paths

Download all attachments as: .zip

Change History (41)

comment:1 by Klumbumbus, 8 years ago

Summary: replace png icons by svg from osm svn source and sort iconsreplace png icons by svg from osm svn source, sort icons and remove duplicates

comment:2 by Klumbumbus, 8 years ago

Milestone: 16.0816.07

Patch is ready, I will commit it today. (This should be in milestone 16.07 to not break external styles/presets for tested users for a long time.)

comment:3 by Klumbumbus, 8 years ago

In 10706/josm:

see #13217 - replace about 166 png icons by svg from https://trac.openstreetmap.org/browser/subversion/applications/share/map-icons/svg reworked most of them (white background or halo, resizing, pixel alignement), draw a few new svg icons from png derived; remove duplicate icons; reorder icons; adapt internal preset and mappaint styles

comment:4 by Klumbumbus, 8 years ago

Some stats:

In images folder there are now:

  • 564 png files
  • 569 svg files
  • 1133 total

For the images/presets folder the numbers are:

  • 162 png files
  • 508 svg files
  • 670 total

comment:5 by bastiK, 8 years ago

Great work!

In the mapcss style, we now rely on the fact that big images get scaled down to 16x16 px. As this value is hardcoded and somewhat arbitrary, I wonder if this could be specified explicitly? Would it help to have a new MapCSS feature for this?

E.g. icon-size: 16; could be used to scale an image proportionally, such that the larger of the two dimensions (width / height) is 16 px.
Or icon-max-size: 16; - the same, only that an image smaller that 16 px remains unscaled.

comment:6 by Klumbumbus, 8 years ago

In 10707/josm:

see #13217 - fix some icons; see #13204 - add reef mappaint style and icon (https://openclipart.org/detail/172292/corals, modified, PD, CC0)

comment:7 by Klumbumbus, 8 years ago

In 10709/josm:

see #13217, see #13204 - fix two icons

comment:8 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:9 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:10 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:11 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:12 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:13 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:14 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:15 by Klumbumbus, 8 years ago

Description: modified (diff)

by Klumbumbus, 8 years ago

Attachment: old_and_new_icon_paths.xlsx added

404 new icon paths

comment:16 by Klumbumbus, 8 years ago

Description: modified (diff)

in reply to:  5 comment:17 by Klumbumbus, 8 years ago

Replying to bastiK:

E.g. icon-size: 16; could be used to scale an image proportionally, such that the larger of the two dimensions (width / height) is 16 px.

Would be nice, so you don't need to care about which one (width or height) is bigger, but I don't see this as high priority.

The more confusing (and error-prone) part is that svg icons between 16 and 64(?) px don't scale down, while bigger ones scale down again. Therefore I resized several icons in r10706.

comment:18 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:19 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:20 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:21 by naoliv, 8 years ago

Sorry if it's already known:

ERROR: Failed to locate image 'presets/car.png'
WARNING:  Car: Could not get presets icon presets/car.png
ERROR: Failed to locate image 'presets/bed.png'
WARNING:  Amenity: Could not get presets icon presets/bed.png
ERROR: Failed to locate image 'presets/shop/mall.png'
WARNING:  Shops: Could not get presets icon presets/shop/mall.png
ERROR: Failed to locate image 'presets/landuse.png'
WARNING:  Landuse: Could not get presets icon presets/landuse.png
ERROR: Failed to locate image 'presets/military.png'
WARNING:  Military: Could not get presets icon presets/military.png
ERROR: Failed to locate image 'presets/firebrigade.png'
WARNING:  Emergency: Could not get presets icon presets/firebrigade.png
ERROR: Failed to locate image 'presets/sport/soccer.png'
WARNING:  Sports: Could not get presets icon presets/sport/soccer.png
ERROR: Failed to locate image 'presets/sightseeing.png'
WARNING:  Tourism: Could not get presets icon presets/sightseeing.png
ERROR: Failed to locate image 'presets/equestrian.png'
WARNING:  Trail Riding Station: Could not get presets icon presets/equestrian.png
ERROR: Failed to locate image 'presets/works.png'
WARNING:  Man Made: Could not get presets icon presets/works.png

The only preset that I have enabled is https://josm.openstreetmap.de/josmfile?page=Presets/NewTags&zip=1 (I am unsure if the missing icons are from NewTags or from something internal in JOSM)

JOSM is 10710

in reply to:  21 comment:22 by Klumbumbus, 8 years ago

Replying to naoliv:

The only preset that I have enabled is https://josm.openstreetmap.de/josmfile?page=Presets/NewTags&zip=1 (I am unsure if the missing icons are from NewTags or from something internal in JOSM)

JOSM is 10710

This is already fixed. It takes up to 10 days until you get the update in your JOSM. To force an update, see https://josm.openstreetmap.de/wiki/Presets#UpdatingpresetsinJOSM

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

comment:23 by naoliv, 8 years ago

Right. Thanks!

comment:24 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:25 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:26 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:27 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:28 by IrlJidel, 8 years ago

Updated Irish boundary presets as requested in http://wiki.openstreetmap.org/wiki/User_talk:IrlJidel

in reply to:  28 comment:29 by Klumbumbus, 8 years ago

Description: modified (diff)

Replying to IrlJidel:

Updated Irish boundary presets as requested in http://wiki.openstreetmap.org/wiki/User_talk:IrlJidel

Thanks.

comment:30 by Klumbumbus, 8 years ago

In 10724/josm:

see #13217, see #13204 - fix icons

comment:31 by Klumbumbus, 8 years ago

In 10725/josm:

see #13217 - fix water_point icon and pixel align aerialway station icon

comment:32 by Michi, 8 years ago

Updated the observatory preset as requested in https://wiki.openstreetmap.org/wiki/User_talk:Michi2#JOSM%20preset

in reply to:  32 comment:33 by Klumbumbus, 8 years ago

Description: modified (diff)

Replying to Michi:

Updated the observatory preset as requested in https://wiki.openstreetmap.org/wiki/User_talk:Michi2#JOSM%20preset

Thanks.

comment:34 by Klumbumbus, 8 years ago

Resolution: fixed
Status: newclosed

Ticket is fixed. 2 github pull requests for external presets/styles are still waiting for merge.

comment:35 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:36 by Klumbumbus, 8 years ago

In 10750/josm:

see #13217 - remove duplicate icon; ignore historic=charcoal_pile 15k uses, but only local usage on 2 spots in the world

comment:37 by Klumbumbus, 8 years ago

In 10751/josm:

see #13217 - fix icon size display for tag combinations like public_transport=stop_position + railway=halt

comment:38 by Klumbumbus, 8 years ago

Description: modified (diff)

comment:39 by simon04, 8 years ago

In 10872/josm:

see #13217 - Update deprecated icon name in TagInfoExtract

comment:40 by simon04, 8 years ago

In 10927/josm:

see #13217 - Only one location for deprecated icon names

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Klumbumbus.
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.