Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop landuse=grave_yard #793

Merged
merged 1 commit into from
Jul 29, 2014

Conversation

matthijsmelissen
Copy link
Collaborator

We still render landuse=cemetery and amenity=grave_yard.

See issue #204.

We still render landuse=cemetery and amenity=grave_yard.

See issue gravitystorm#204.
@gravitystorm gravitystorm self-assigned this Jul 29, 2014
gravitystorm added a commit that referenced this pull request Jul 29, 2014
@gravitystorm gravitystorm merged commit 5b411de into gravitystorm:master Jul 29, 2014
@matthijsmelissen matthijsmelissen deleted the graveyard branch August 6, 2014 23:45
@matkoniecz
Copy link
Contributor

Note that names on landuse=grave_yard are still rendered:

area yes landuse grave_yard name a _19_closed_way

@matthijsmelissen
Copy link
Collaborator Author

Yes, that deserves a new issue.

@dieterdreist
Copy link

Sorry for joining this discussion so late. I have been misled by the text
"landuse=grave_yard" and ignored this but have now learnt that the
intention seems to be removing "amenity=grave_yard" from the rendering as
well? Please do not do this. A cemetery and a grave yard are not
necessarily the same thing: a cemetery is a dedicated place to bury dead
people, typically without a main church (there could be a small church or a
chapel, but they are typically there for funeral ceremonies only). On the
other hand, a grave yard can be a cemetery, but it can also be a church
yard (a cemetery cannot be a church yard!), which is a place around a
church, used for burying people (typically the more important people, as
space is limited). These are much older features (historically) than
cemeteries.

This aside, "landuse" is IMHO not a nice tag for neither cemeteries nor
grave yards (would prefer amenity for all of these), but that should be
discussed on tagging ML...

@matkoniecz
Copy link
Contributor

There is no reason to drop rendering amenity=grave_yard - and I see no indication of "removing "amenity=grave_yard" from the rendering as well".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants