Modify

Opened 21 months ago

Last modified 21 months ago

#22793 new enhancement

[PATCH] Render boundary=place

Reported by: JeroenHoek Owned by: team
Priority: normal Milestone:
Component: Internal mappaint style Version:
Keywords: boundary place Cc:

Description (last modified by JeroenHoek)

Render boundary=place as a boundary. This tag originated in the Netherlands for use with boundaries of defined places (neighbourhoods and quarters) which lack administrative boundaries, but have formal boundaries nonetheless.

The tag is now in broad use all over the world with nearly 8000 instances, including in Estonia, Georgia, and Hell's Kitchen in New York.

The second patch adds the accompanying preset. It just mirrors what is done for the other boundaries, and thus uses the same icon as the rest.

Attachments (2)

0001-Add-boundary-place-rendering.patch (1.3 KB ) - added by JeroenHoek 21 months ago.
0002-Add-boundary-place-preset.patch (1.8 KB ) - added by JeroenHoek 21 months ago.

Download all attachments as: .zip

Change History (6)

comment:1 by skyper, 21 months ago

Keywords: boundary place added

Well, Key:place is already allowed on areas. Why do we need Tag:boundary=place? Seems to be an unneeded duplicate in my eyes. I only find discussions on a local level (Netherlands). Was there any global discussion?

Overall, I prefer to have a default preset for major tags present in core mappaint which often comes down to find or create an appropriate icon.

comment:2 by JeroenHoek, 21 months ago

It is used almost 8000 times, is well documented, and has been extensively discussed with consensus sought and found. The wiki-page even shows an example in Belgium.

This is really not the place to discuss the merits or demerits of this tag.

Overall, I prefer to have a default preset for major tags present […]

This is not a preset, just styling that prevents mappers from seeing a meaningless grey line. That helps prevent mappers making mistakes, and makes their mapping experience more pleasant.

Last edited 21 months ago by JeroenHoek (previous) (diff)

comment:3 by JeroenHoek, 21 months ago

Description: modified (diff)

by JeroenHoek, 21 months ago

comment:4 by JeroenHoek, 21 months ago

Description: modified (diff)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to JeroenHoek.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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