Opened 8 years ago

Closed 5 years ago

#3885 closed defect (wontfix)

[landcover] landuse=military covers leisure=pitch

Reported by: flaimo Owned by: mapnik-team@…
Priority: minor Milestone:
Component: mapnik Version:
Keywords: Cc:

Description

the (ugly) lines from landuse=military cover areas with leisure=pitch, sports=x.

see: http://osm.org/go/0JhLXApnD-

Change History (6)

comment:1 Changed 8 years ago by Tom Hughes

Component: adminmapnik
Owner: changed from Tom Hughes to mapnik-team@…

comment:2 Changed 8 years ago by bbauer

I think both ways (painting the pitch above the area marks of the military area or the other way round) are reasonable and have their advantages. I think the actual way is more logical for the following reason:

The pitch is part of the area that has military landuse. As far as I know military areas are typically not accessible by the public and therefore also the pitch is not acessible. This is shown by the light red overlay and the red "strike-out".

If you would allow the pitch to berendered above the military style it would look like it was not part of the surrounding military area.

comment:3 Changed 8 years ago by flaimo

i don't agree with that: a) landuse describes something logical, leisure=pith describes something physical. b) the style should be consistent. other areas don't draw over leisure=pitch, so why should landuse=military. for example landuse=allotments also count as private property, still leisure=pitch is drawn above the landuse.

comment:4 Changed 8 years ago by flaimo

to precise point a: the kind of access should be determinated by the access tag on the actual element, not by the landuse below.

comment:5 Changed 5 years ago by math1985

Summary: landuse=military covers leisure=pitch[landcover] landuse=military covers leisure=pitch

comment:6 Changed 5 years ago by math1985

Resolution: wontfix
Status: newclosed

In this case, the pitch is actually part of the military landuse, so I think the current rendering is correct.

I will therefore close this ticket as wontfix.

Note: See TracTickets for help on using tickets.