Modify

Opened 3 months ago

Last modified 3 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 3 months ago.
0002-Add-boundary-place-preset.patch (1.8 KB) - added by JeroenHoek 3 months ago.

Download all attachments as: .zip

Change History (6)

Changed 3 months ago by JeroenHoek

comment:1 Changed 3 months ago by skyper

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 Changed 3 months ago by JeroenHoek

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 3 months ago by JeroenHoek (previous) (diff)

comment:3 Changed 3 months ago by JeroenHoek

Description: modified (diff)

Changed 3 months ago by JeroenHoek

comment:4 Changed 3 months ago by JeroenHoek

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.
to The owner will be changed from team to the specified user.
The owner will change to JeroenHoek
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.