Modify

Opened 4 years ago

Closed 3 years ago

Last modified 3 years ago

#20835 closed enhancement (fixed)

Add/ignore popular tags: attraction=water_slide, designation=public_bridleway/public_footpath, ownership=private, wall=brick/stone, water=ditch/stream

Reported by: Don-vip Owned by: team
Priority: normal Milestone: 21.07
Component: Internal preset Version:
Keywords: Cc: Klumbumbus

Description (last modified by Klumbumbus)

Attachments (4)

josm_20835_wall_water.patch (4.4 KB ) - added by skyper 3 years ago.
patch adding wall=brick|stone and water=stream|ditch to defaultpresets and some icons for the combo
josm_20835_ignoretags.cfg.patch (510 bytes ) - added by skyper 3 years ago.
patch ignoring "ownership" and "designation"
josm_20835_presets_without_icons_v2.patch (4.0 KB ) - added by skyper 3 years ago.
patch for amenity=events_venue and boundary=marker. Own icons missing.
200b.png (23.3 KB ) - added by Don-vip 3 years ago.

Download all attachments as: .zip

Change History (33)

comment:1 by skyper, 4 years ago

ownership comes mainly from imports and designation looks like a key almost exclusively used in Great Britain plus already included in Presets/EnglandWalesRightsOfWay. Thus I tend to ignore these tags for now.

The rest could be added to defaultpresets and default style, I guess.

comment:2 by simon04, 3 years ago

Milestone: 21.0521.06

by skyper, 3 years ago

Attachment: josm_20835_wall_water.patch added

patch adding wall=brick|stone and water=stream|ditch to defaultpresets and some icons for the combo

comment:3 by skyper, 3 years ago

We need an icon for water_slide which is different to leisure=water_park: source:trunk/resources/images/presets/leisure/water_park.svg

For wall=brick|stone and water=stream|ditch the fix is easy and no style update is needed. Please, find attached patch josm_20835_wall_water.patch which additionally adds some icons to the combos.

by skyper, 3 years ago

patch ignoring "ownership" and "designation"

comment:4 by Don-vip, 3 years ago

Milestone: 21.0621.07

comment:5 by Don-vip, 3 years ago

In 18032/josm:

see #20835 - add wall=brick|stone, water=stream|ditch, ignore "ownership" and "designation", add some icons in comboboxes (patches by skyper)

comment:6 by Don-vip, 3 years ago

thanks!

comment:7 by Don-vip, 3 years ago

Description: modified (diff)

comment:8 by Don-vip, 3 years ago

Description: modified (diff)

comment:9 by Don-vip, 3 years ago

In 18033/josm:

see #20835 - add shelter_type=gazebo (needs its own icon), ignore cai_scale, building=government

comment:10 by Don-vip, 3 years ago

Description: modified (diff)

comment:11 by Don-vip, 3 years ago

Description: modified (diff)

comment:12 by Don-vip, 3 years ago

Cc: Klumbumbus added

I've done the quick and easy updates. Remaining: those which needs some research/icons/new presets

comment:13 by skyper, 3 years ago

I am unsure about ​boundary=local_authority as it was not approved (just "in use") and looks like a value for only certain countries.

comment:14 by Don-vip, 3 years ago

It's widely used in France and emerged from a correct community discussion/consensus. But agreed, it's not worldwide (although I see India followed our proposal). Better to include it in French/Indian specific presets, as the values possible for each country will never be the same.

comment:15 by Klumbumbus, 3 years ago

In 18090/josm:

see #20835 - Add own icon for shelter_type=gazebo (icon derived from leisure/bandstand.svg, CC0 and PD licensed)

comment:16 by Klumbumbus, 3 years ago

Description: modified (diff)

comment:17 by Klumbumbus, 3 years ago

In 18091/josm:

see #20835 - Enhance location combo boxes (add overhead), remove one unneeded chunk

comment:18 by Klumbumbus, 3 years ago

Description: modified (diff)

comment:19 by Klumbumbus, 3 years ago

In 18092/josm:

see #20835 - Add attraction=water_slide, use icon from leisure=water_park, for leisure=water_park use a new icon instead: https://wiki.openstreetmap.org/wiki/File:Swimming-16.svg (PD and CC0 licensed)

comment:20 by Klumbumbus, 3 years ago

Description: modified (diff)

by skyper, 3 years ago

patch for amenity=events_venue and boundary=marker. Own icons missing.

comment:21 by skyper, 3 years ago

josm_20835_presets_without_icons_v2.patch covers the last two missing presets, amenity=events_venue and boundary=marker, but lacks icons, so far.

comment:22 by Klumbumbus, 3 years ago

In 18094/josm:

see #20835 - Add boundary=marker and amenity=events_venue (patch by skyper), icons derived from service/community_centre and landmark/boundary_stone, both PD and CC0 licensed, ignore boundary=local_authority for now

comment:23 by Klumbumbus, 3 years ago

Description: modified (diff)
Resolution: fixed
Status: newclosed

in reply to:  22 ; comment:24 by Klumbumbus, 3 years ago

Replying to Klumbumbus:

In 18094/josm:

... ignore boundary=local_authority for now

I don't know why Jenkins still warns about boundary=local_authority.

comment:25 by skyper, 3 years ago

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

by Don-vip, 3 years ago

Attachment: 200b.png added

in reply to:  24 ; comment:26 by Don-vip, 3 years ago

Replying to Klumbumbus:
There's an invisible character between : and b:


comment:27 by Don-vip, 3 years ago

In 18101/josm:

see #20835 - remove invisible U+200B whitespace character

comment:28 by Don-vip, 3 years ago

In 18102/josm:

see #20835 - ignore healthcare=yes

in reply to:  26 comment:29 by Klumbumbus, 3 years ago

Replying to Klumbumbus:
There's an invisible character between : and b:

Thanks!

Modify Ticket

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