Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#10835 closed defect (fixed)

consistency between preset and default mappaint style — at Version 41

Reported by: Klumbumbus Owned by: Klumbumbus
Priority: normal Milestone: 15.01
Component: Internal mappaint style Version:
Keywords: Cc:

Description (last modified by Klumbumbus)

This ticket can be used to refer to, when changes are made regarding to this topic.

The goal is:

  1. same icon for one feature
  2. support of the same geometries (node,way,closedway,relation)

in preset and mappaint style.

Also every item in the preset menu should have its own icon.

Deprecated icons should be moved to images_nodist folder.

So far the following changes have already been made:

[7717] - landuse=orchard, barrier=spikes, man_made=groyne, man_made=breakwater
[7711] - landuse
[7724] - sport
[7728] - generator:source
[7729] - golf
[7735] - natural
[7738] - amenity=drinking_water
[7744] - restriction
[7668][7736][7739] - move deprecated icons to images_nodist/presets folder

related osm svn changes

...more will come...

Change History (42)

comment:1 by Klumbumbus, 11 years ago

Description: modified (diff)

comment:2 by bastiK, 11 years ago

TODO: barrier=bollard

comment:3 by Klumbumbus, 11 years ago

In 7821/josm:

see #10835 - icon unification and geometry consistency for barrier=*

comment:4 by Klumbumbus, 11 years ago

In 7854/josm:

see #10835 - icon unification and geometry consistency for highway waypoints

comment:5 by Klumbumbus, 11 years ago

In 7877/josm:

see #10835 - remove highway=unsurfaced from mapaint style

comment:6 by Klumbumbus, 11 years ago

Milestone: 14.1215.01

There is still some work to do. I will do it in next milestone.

comment:7 by Klumbumbus, 11 years ago

todo: aerialway=t-bar

comment:8 by Don-vip, 11 years ago

You can do it now if you want :)

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

Replying to Don-vip:

You can do it now if you want :)

No, I want to do it in the next days and check the whole aerialway group. This was just a reminder, that I do not forget this :)

comment:10 by Don-vip, 11 years ago

allright!

comment:11 by Don-vip, 11 years ago

Description: modified (diff)

comment:12 by Klumbumbus, 11 years ago

In 7915/josm:

see #10835 - icon unification and geometry consistency for aerialway

comment:13 by Klumbumbus, 11 years ago

Ticket #10914 has been marked as a duplicate of this ticket.

comment:14 by Klumbumbus, 11 years ago

In 7927/josm:

see #10835 - icon unification and geometry consistency for whole facilities preset group

comment:15 by aceman, 11 years ago

amenity=ice_cream has no preset, but has an icon if created as a node.

comment:17 by Klumbumbus, 11 years ago

In 7940/josm:

see #10835 - icon unification for place=*

comment:18 by Klumbumbus, 11 years ago

In 7942/josm:

see #10835 - icon unification and geometry consistency for some naturaltags. add validator warnings for some node features on ways

comment:19 by Klumbumbus, 11 years ago

In 7950/josm:

see #10835 - icon unification and geometry consistency for some shop tags. add own icon for shop=alcohol

comment:20 by rickmastfan67, 11 years ago

Some icons got broken in the [7950] changeset. Saw this in the CL in Windows 7 with the newest 'latest'.

ERROR: Failed to locate image 'styles/standard/shop/rental/library.png'
WARNING: Could not get presets icon styles/standard/shop/rental/library.png

comment:21 by Klumbumbus, 11 years ago

In 7952/josm:

see #10835 - fix icon path

in reply to:  20 comment:22 by Klumbumbus, 11 years ago

Replying to rickmastfan67:

Some icons got broken in the [7950] changeset.

Thanks. Fixed.

comment:23 by Klumbumbus, 11 years ago

In 7953/josm:

see #10835 - new icon for shop=books

comment:24 by Klumbumbus, 11 years ago

In 7960/josm:

see #10835 - icon unification and geometry consistency for some man_made, historic amd bridge tags.

comment:25 by Klumbumbus, 11 years ago

Description: modified (diff)

comment:26 by Klumbumbus, 11 years ago

Description: modified (diff)

comment:27 by Klumbumbus, 11 years ago

In 7962/josm:

see #10835 - move deprecated icons to nodist, see [o30914]

comment:28 by Klumbumbus, 11 years ago

In 7963/josm:

see #10835 - fix regression of r7960

in reply to:  24 comment:29 by rickmastfan67, 11 years ago

Replying to Klumbumbus:

In 7960/josm:

see #10835 - icon unification and geometry consistency for some man_made, historic amd bridge tags.

Missed one. Saw this in the CL on startup of the new 'latest' r7963.

ERROR: Failed to locate image 'styles/standard/bridge/support.svg'
WARNING: Could not get presets icon styles/standard/bridge/support.svg

comment:30 by Klumbumbus, 11 years ago

In 7964/josm:

see #10835 - fix typo

comment:31 by Klumbumbus, 11 years ago

In 7967/josm:

see #10835 - icon unification and geometry consistency for some power tags. add validator warnings

in reply to:  31 ; comment:32 by skyper, 11 years ago

Replying to Klumbumbus:

In 7967/josm:

see #10835 - icon unification and geometry consistency for some power tags. add validator warnings

Better use https://wiki.openstreetmap.org/wiki/Tag:power=substation#Switchgear and no redirects as redirects do not support languages and the wiki already includes four languages besides English.

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

Replying to skyper:

Better use https://wiki.openstreetmap.org/wiki/Tag:power=substation#Switchgear and no redirects as redirects do not support languages and the wiki already includes four languages besides English.

But this would also not work for russian and polish, since they do not have a heading "Switchgear". And if one day someone creates the page power=switchgear, we will likely not notice for the JOSM preset and change power=substation#Switchgear to power=switchgear.

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

in reply to:  33 ; comment:34 by skyper, 11 years ago

Guess we do not get around in these cases to do the job manually for each language as the script to automatically check/include the links will fail here, too.

comment:35 by Klumbumbus, 11 years ago

In 7975/josm:

see #10835 - icon unification and geometry consistency for some water related tags. add validator warnings

in reply to:  34 comment:36 by Klumbumbus, 11 years ago

Replying to skyper:

Guess we do not get around in these cases to do the job manually for each language as the script to automatically check/include the links will fail here, too.

I think the script will override manual edits each time it is executed.

comment:37 by Klumbumbus, 11 years ago

In 7977/josm:

see #10835 - icon unification and geometry consistency for some transport related tags (railway, aeroway,...). add validator warnings

comment:38 by Klumbumbus, 11 years ago

In 7978/josm:

see #10835 - fix hangar

comment:39 by Klumbumbus, 11 years ago

Resolution: fixed
Status: newclosed

This is all I plan to do for now. Public transport is missing. I created an own ticket for this #11006.

Please report here or create new tickets, if you notice more inconsistecies between preset/mappaint/validator/wiki.

comment:40 by bastiK, 11 years ago

Great work, thanks!

comment:41 by Klumbumbus, 11 years ago

Description: modified (diff)

by skyper, 11 years ago

Attachment: historic_panel.png added

screenshot

Note: See TracTickets for help on using tickets.